Re: [xwiki-devs] [Proposal] Having the AppBar panel in the left side column by default.

On 29 May 2014 at 11:23:01, Marius Dumitru Florea ( mariusdumitru.florea@xwiki.com(mailto:mariusdumitru.florea@xwiki.com)) wrote: have would be because I Note that Guillaume forgot to mention that the icons will be larger (see http://design.xwiki.org/xwiki/bin/download/Improvements/Skin4x/homepageDesk…). Right now they are the ones from the panel in medium width mode because Guillaume has implemented it yet. @Guillaume: regarding icon colors, what would be your plan? Blue and white colors as on http://design.xwiki.org/xwiki/bin/download/Improvements/Skin4x/homepageDesk… even for the Colibri skin or different coloring depending on the skin? I also think that it might be best to not change Colibri since people are probably used to it for now. Now we need to find a way to have panels defined differently for each skin for that.

There are various solutions for this: * Solution 1: Add support for storing configuration (XWikiPreferences) in skin pages and modify the DefaultConfigurationSource to be profile > skin > space > wiki or possibly better: profile > space > wiki > skin * Solution 2: Refactor the UI configuration in the Admin UI by moving all UI-related configuration parameters in some UIConfig xobjects that depend on the selected skin (one xobject per skin). Make the skin selection the first item on the page and when you change it the rest of the UI configuration below reflects the content of the UIConfig page corresponding to that skin. This means introduction a UIConfigurationSource component impl but that’s easy to do. * Solution 3: Modify the leftpanels/rightpanels configuration properties from String to TextArea and allow scripting to be put inside. Then, by default, have some “if” to handle both Colibri and Flaming default panels. When the user uses the Panel Wizard the first time, the value will be overridden by the list of panels selected by the user. * Solution 4: In the DefaultSkin page, override leftpanels.vm/leftpanels.vm and check if the user has defined any panel preferences in XWikiPreferences. If so, then use them. If not, then compute the left and right panels default values. There are obviously downsides to all these solutions... Personally I think I prefer solution 1 with profile > space > wiki > skin, i.e. if the leftpanels/rightpanels xproperties are not defined at the profile, space, wiki levels (and by default we should leave it empty there) then the value is taken from the skin configuration component. We would need to define how the Panel Wizard works (we probably have an issue even now when the panels are defined in a user’s profile). It should probably be possible int he Panel Wizard UI to select what level is being configured (wiki, space, profile, skin). That’s the main issue for which we need a solution: that it is not too magical for the user when they go to the Panel Wizard. More generally speaking a we have an issue in the Admin UI. We should probably show the inferred values (real value used) and indicate the value for a given property at all levels (wiki, space, profile, skin) and allow users to easily change the value for the level they wish. We need some more brainstorming about this. Any other ideas or variations? Thanks -Vincent _______________________________________________ devs mailing list devs(a)xwiki.org http://lists.xwiki.org/mailman/listinfo/devs