[OPT-1356] Improve UI of view, groupBy and visualization option selection for report creation Created: 23/Jul/18  Updated: 18/Apr/19  Resolved: 01/Aug/18

Status: Done
Project: Camunda Optimize
Component/s: frontend
Affects Version/s: None
Fix Version/s: 2.2.0-alpha1, 2.2.0

Type: Task Priority: L3 - Default
Reporter: Sebastian Stamm Assignee: Unassigned
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File Screen Shot 2018-08-01 at 10.43.23.png    
Issue Links:
Related
is related to OPT-1376 Make group by variables ui its own co... Open

 Description   

AT:

  • The select components of the report control panel are replaced with dropdown components
  • The view dropdown contains submenus for Frequency Count, Process Instance Duration and Flownode Duration
  • The group by dropdown contains submenus for Start Date and Variables
  • The Variables submenu follows the specification found at https://zpl.io/Vk51n08
  • It is still impossible to create invalid combinations, by disabling certain dropdown entries and providing an autocomplete wizard for unambiguous choices


 Comments   
Comment by Omran Abazeed [ 30/Jul/18 ]

In term functionality, I believe it is working correctly according to my testing but it was very hard to understand what is happening
I have a few comments :

  • The dropdown menu could be displayed on hover instead of click, I think it will be more natural this way
  • I suggest hiding menu items that are not possible instead of disabling them for simpler menus
  • regarding AT 4: I have to log in to see the specification
  • I find the methods in reportConfig very complex and very hard to maintain, I do not know if there is any way to simplify or maybe documenting all the possible situations that could occur
  • in reportConfig.js I suggest to create an object from the variables line 35-53 and include the line 59 also in the object, and then maybe call it something like groupToVizLinks. It was confusing to me at first especially the line 59
  • I do not if that is done already but I suggest creating a new ticket for separating the variable dropdown into another component
Comment by Sebastian Stamm [ 31/Jul/18 ]

For your first two points, I would suggest talking to Franz as this is an UX topic and not only an implementation issue. We can do so in the team meeting and create new tickets if we want to change any behavior.

regarding AT 4: I have to log in to see the specification

I think Franz can set this up for you.

I find the methods in reportConfig very complex and very hard to maintain, I do not know if there is any way to simplify or maybe documenting all the possible situations that could occur.

I tried to document them via the tests. So for every scenario where these methods may be used there should be a test showing the input and output. If you have any other idea how we can improve it, I am happy to discuss and implement that

in reportConfig.js I suggest to create an object from the variables line 35-53 and include the line 59 also in the object, and then maybe call it something like groupToVizLinks. It was confusing to me at first especially the line 59

Good idea; I created an object called combinations for that.

I do not if that is done already but I suggest creating a new ticket for separating the variable dropdown into another component

I created OPT-1376 for that.

Comment by Franz Heidl [ 01/Aug/18 ]

A couple of things that need fixing or looking into:

  • The input in the top right (typeahead) has a visible margin to the right. There is no width defined for the input, I suggest setting the width of the input to 100% in the context of its container.
  • the font-size in the menu is set to 1em which results in 16px, should be 14px.
  • the checkmark denoting a selected entry seems to hang too low in relation to the text labels, this might be ok when the font-size has been fixed.
  • I have added the svg files for the arrow icon (left, up, right, down) to the project, please use these.
Comment by Sebastian Stamm [ 08/Aug/18 ]

Franz Heidl: Thanks for the feedback! I created a new ticket to implement the suggested changes: OPT-1392

Generated at Sun Oct 20 05:56:40 CEST 2019 using JIRA 6.4.6#64021-sha1:33e5b454af4594f54560ac233c30a6e00459507e.