@pebosi opened this Issue on August 17th 2009 Contributor

This patch outlines the two topBars with a small border and a background-image.

@pebosi commented on August 17th 2009 Contributor
@pebosi commented on August 17th 2009 Contributor

Attachment:
piwik-941.patch

@mattab commented on August 17th 2009 Owner

I am not sure about this UI modification. It seems to outline the top bar, which is a rarely used UI element and might distract users from focusing on the reports. What do others think?

@robocoder commented on August 18th 2009 Contributor

I'm ok with adding the topbar div as a container. The css changes could be deferred to future themeability...

@mattab commented on August 18th 2009 Owner

pebosi, if you are interested, you can have a look at what it would take to add theming in Piwik. I'll forward you an email.

It would be nice to leave the design of Piwik the same for the next few releases and try to add the theming feature in Core. Then themes could freely modify the look of the app.

@robocoder commented on August 18th 2009 Contributor

In [1410], add #topBars div; css changes deferred

This Issue was closed on August 18th 2009
Powered by GitHub Issue Mirror