Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #3396] Revamped cronk list menu, list and icon style, scroll behavior #950

Closed
icinga-migration opened this issue Oct 24, 2012 · 15 comments
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Oct 24, 2012

This issue has been migrated from Redmine: https://dev.icinga.com/issues/3396

Created by mfriedrich on 2012-10-24 18:01:07 +00:00

Assignee: mfrosch
Status: Resolved (closed on 2013-04-19 14:46:59 +00:00)
Target Version: 1.9
Last Update: 2013-04-19 14:46:59 +00:00 (in Redmine)


Improving the design and behavior of the cronk menu


old description

some people prefer it more static (like the classic ui), so the clickable icons might confuse people away.

my proposal - add simple static links to the left menu, basically all categories open, but allowing to hide them on demand (like classic ui).

i am not sure if that fits with the current fancy layout, but it would maybe help in acceptance when moving from old to new.

Attachments

Changesets

2012-11-29 09:19:27 +00:00 by (unknown) 17b55c9

New cronk list style, icon style still selectable

CSS styles updated
CronkListingPanel updated to allow switching (with states)
added config option org.icinga.cronk.liststyle (list|icon)

(refs #3396)

2013-03-13 16:28:08 +00:00 by mfrosch 23926e4

Improve scroll behavior of Cronk menu (refs #3396)

* better scrolling when many categories
* scrolling based on whole many, not box contents
* added background to avoid ugly white box

2013-03-13 16:31:38 +00:00 by mfrosch 6f0868b

Further improve look of new Cronk list menu (refs #3396)

* no italic - hover looks cleaner
* little bit bigger icons
* items are a bit higher as well
* avoid wrapping

2013-04-03 14:01:55 +00:00 by mfrosch 54199ce

Fix glitches with the icon Cronk menu (refs #3396)

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 25, 2012

Updated by pdeneu on 2012-10-25 12:59:49 +00:00

i think it would be even better to get option in user menu to select if icons or text are displayed.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Oct 25, 2012

Updated by mfriedrich on 2012-10-25 13:11:02 +00:00

or that one.

though, i was thinking of how the extjs menu is done in various demos - there's also sort of a tree view possible, but i doubt that this would fit for the general menue - unless the cronk "categories" would support grouping there as well (dynamic).

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Nov 29, 2012

Updated by mfrosch on 2012-11-29 09:24:38 +00:00

  • Assigned to set to mfrosch
  • Target Version changed from 1.10 to 1.9

Done in my branch mfrosch/ui-improvement

See Screenshort attached

New cronk list style, icon style still selectable

CSS styles updated
CronkListingPanel updated to allow switching (with states)
added config option org.icinga.cronk.liststyle (list|icon)

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Nov 29, 2012

Updated by mfrosch on 2012-11-29 09:25:14 +00:00

  • File added 59.png
  • Category set to Cronks
  • Done % changed from 0 to 80
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Nov 29, 2012

Updated by mfrosch on 2012-11-29 09:27:10 +00:00

  • File added 59.png
  • Status changed from New to Feedback

Another screenshot

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Feb 22, 2013

Updated by mfrosch on 2013-02-22 16:39:35 +00:00

  • Status changed from Feedback to Resolved

Merged into master

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Feb 22, 2013

Updated by mfrosch on 2013-02-22 16:40:11 +00:00

  • Done % changed from 80 to 100
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 1, 2013

Updated by mfriedrich on 2013-03-01 20:29:55 +00:00

  • Status changed from Resolved to 7
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 1, 2013

Updated by mfriedrich on 2013-03-01 20:39:06 +00:00

since this introduces a change in behaviour to the gui, i'd like to hear more opinions on that.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 3, 2013

Updated by mfriedrich on 2013-03-03 11:13:27 +00:00

  • File added icinga_web_1.8_menu_big_icons_full_screen_1920x1080.png
  • File added icinga_web_1.9_upgrade_from_1.8_no_appstate_reset_menu_width_too_small.png
  • File added icinga_web_1.9_upgrade_from_1.9_reset_appstate.png
  • File added icinga_web_1.9_menu_small_icons_list_full_screen_1920x1080.png

there's a problem when upgrading from r1.8 to next (say "1.8.x to 1.9.x"), having that change of the menu enabled.

if you do not reset the application state for the user, the width of the menu is smaller (like the small icon list would need), but the bigger icons are still shown, and somewhat disordered.

this should be taken into account when proposing this change to users, especially when support questions may turn in.

other than that, i still like the new view.

r1.8 (1.8.2) with big icons menu

icinga_web_1.8_menu_big_icons_full_screen_1920x1080.png

next (pre 1.9.0) with current appstate after upgrade

icinga_web_1.9_upgrade_from_1.8_no_appstate_reset_menu_width_too_small.png

next (pre 1.9.0) reset appstate for the user

icinga_web_1.9_upgrade_from_1.9_reset_appstate.png

next (pre 1.9.0) with small icons list

icinga_web_1.9_menu_small_icons_list_full_screen_1920x1080.png

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 3, 2013

Updated by mfriedrich on 2013-03-03 11:17:46 +00:00

  • Status changed from 7 to Assigned
  • Done % changed from 100 to 90

maybe the appstate reset can be resolved somehow, at least it should be mentioned in the upgrade docs. the docs issue should be opened once the direction (which menue is the default?) to go is clear.

it should contain

  • menue screenshots for the official docs and a description of the 2 menues available
  • a description with a config snippet how to change behaviour
  • upgrading docs need the appstate reset, if not resolved earlier
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Mar 13, 2013

Updated by mfrosch on 2013-03-13 16:28:01 +00:00

  • Subject changed from Replace the cronk icons with a list, seperated by category to Revamped cronk list menu, list and icon style, scroll behavior
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 3, 2013

Updated by mfrosch on 2013-04-03 14:37:28 +00:00

https://www.icinga.org/2013/03/19/vote-on-your-future-icinga-web-cronk-menu/

TODO: Update Docs, Check config for comments on the setting

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 12, 2013

Updated by mfriedrich on 2013-04-12 16:48:15 +00:00

the docs are uptodate on docs git 'next' including updated images and the change setting. which configuration do you mean? might require some docs text too.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 19, 2013

Updated by mfriedrich on 2013-04-19 14:46:59 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 90 to 100

see #3983 for config option and further docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.