Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New setting: enable Left Menu mode #5212

Closed
mattab opened this issue May 23, 2014 · 15 comments
Closed

New setting: enable Left Menu mode #5212

mattab opened this issue May 23, 2014 · 15 comments
Assignees
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Milestone

Comments

@mattab
Copy link
Member

mattab commented May 23, 2014

As a Piwik user, I want to see the main menu displayed on the left.

Advantages of Left Menu:

  • left menu may be easier to use for some people.
  • able to display more pixels of data from top to bottom, which may avoid some day to day scrolling
  • It scales better when more sub-menus are added.
  • Left menu is a preference only. Zen mode will still use its own characteristic top menu design.

Ideas

  • new setting to enable Left Menu
  • super user can set default for all users that haven't chosen their setting yet
  • Plugin could be enabled by default but the setting would be disabled by default. I think this would be a better user experience since it also allows people not having super user access to enable the setting and are not dependent on the super user
  • users can enable/disable
  • make it a plugin not a theme. This setting will automatically work for all Themes. Nice.
  • Ideally there would be a setting to enable it only for large displays. Eg on tablets or mobile phones it does not make sense to have it on the left side. On the other side on large displays it makes sense. We could detect the screen size and enable the left menu client side. Would most likely also fix the problem that we do not have to regenerate the CSS all the time and we would get rid of some CSS !important definitions.
@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 62acfde: refs #5212 improved left menu stylesheet which works better with Morpheus now, turned it into a plugin and made it configurable. not finished yet

@tsteur
Copy link
Member

tsteur commented May 29, 2014

Faced some challenges here so far:

  • As each user can enable/disable the LeftMenu the generated CSS is different for each user. If two or more users are using the same Piwik instance at the same time this results basically in regenerating the CSS all the time. This is a general issue and most likely needs a separate ticket. We could for instance persist different CSS files for each cacheBuster value: asset_....css$CB.css . Meaning we serve the correct CSS file to the correct users and do not have to recreate them all the time.
  • A SystemSetting is currently only read AND writable by super users. Meaning I cannot access the default value whether by default the left menu is enabled or not if a user is not the super user. We need either a new type of SystemSetting which allows to READ for everyone but write only for SuperUsers or we modify the existing one. Not sure yet

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 82da315: refs #5212 added possibility to make a system setting readable for non super users

@tsteur
Copy link
Member

tsteur commented May 29, 2014

Found a solution for SystemSetting which is backwards compatible and prevents us from creating lots of classes as some SystemSettings should be maybe readable by anonymous users whereas some should be readable only for users having some view access etc. This would be also a proper solution for this hack: https://github.com/piwik/piwik/blob/master/core/Plugin/Settings.php#L330 ( #4924 )

To be defined whether we remove this hack

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In bb4d352: refs #5212 do not display the default value in case we handle radio fields

@tsteur
Copy link
Member

tsteur commented May 29, 2014

Ideally there would be a setting to enable it only for large displays. Eg on tablets or mobile phones it does not make sense to have it on the left side. On the other side on large displays it makes sense. We could detect the screen size and enable the left menu client side. Would most likely also fix the problem that we do not have to regenerate the CSS all the time and we would get rid of some CSS !important definitions.

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 15f3cad: refs #5212 this should enable leftMenu plugin by default but it will be disabled. Need to test it

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 5bbf3b7: refs #5212 added some tests

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 7d1b664: refs #5212 should fix core tests

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 8fb7b4c: refs #5212 always add leftMenu css but apply it only if there is a class .leftMenuPlugin. Added an event to add classes to the body which I found quite useful already for a while

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 4e950e8: refs #5212 a few ui tweaks

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 859a682: refs #5212 css => less

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 0134372: refs #5212 translations

@tsteur
Copy link
Member

tsteur commented May 29, 2014

In 9a960ae: refs #5212 another improvement for combo of zenMode + leftMenu

@mattab
Copy link
Member Author

mattab commented Jul 2, 2014

In 8208b89: Refs #5212 add missing reference, this maybe was a merge-fail

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Projects
None yet
Development

No branches or pull requests

2 participants