Opened 5 years ago

Closed 5 years ago

#941 closed New feature (fixed)

Outline topBars

Reported by: pebosi Owned by:
Priority: major Milestone: Piwik 0.4.4
Component: UI - UX (AngularJS, twig, less) Keywords:
Cc: Sensitive: no

Description

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

Attachments (2)

topbars-screenshot.png (12.6 KB) - added by pebosi 5 years ago.
piwik-941.patch (1.6 KB) - added by pebosi 5 years ago.

Download all attachments as: .zip

Change History (7)

Changed 5 years ago by pebosi

Changed 5 years ago by pebosi

comment:1 Changed 5 years ago by vipsoft (robocoder)

  • Milestone changed from Features requests - after Piwik 1.0 to 1 - Piwik 0.4.4

comment:2 Changed 5 years ago by matt (mattab)

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?

comment:3 Changed 5 years ago by vipsoft (robocoder)

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

comment:4 Changed 5 years ago by matt (mattab)

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.

comment:5 Changed 5 years ago by vipsoft (robocoder)

  • Resolution set to fixed
  • Status changed from new to closed

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

Note: See TracTickets for help on using tickets.