Ticket #726 (closed Bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Wordpress CSS bug

Reported by: black silence Owned by:
Priority: major Milestone: RobotRock
Component: Website piwik.org Keywords:
Cc: Sensitive:

Description

The piwik.org wordpress stylesheet does not define a background color. Patch attached, please fix.

Attachments

style.patch Download (0.6 KB) - added by black silence 3 years ago.
screenshot8.png Download (25.2 KB) - added by black silence 3 years ago.
Screenshot of problem

Change History

Changed 3 years ago by black silence

Changed 3 years ago by black silence

Screenshot of problem

Changed 3 years ago by vipsoft

  • status changed from new to closed
  • resolution set to invalid

I think you need to pass this along to the WP plugin author.

Piwik does set the background colour in themes/default/simple_structure.css

Changed 3 years ago by feyp

vipsoft: This is not about the WP plugin and not about Piwiki. This is about the piwik.org website (see the screenshot).

Changed 3 years ago by vipsoft

  • status changed from closed to reopened
  • resolution invalid deleted

sorry too early in the a.m.

what browser are you using?

Changed 3 years ago by feyp

No problem.

I'm not the reporter of the bug, but you can reproduce the bug in any browser. Just set the default background colour to something else than white. The patch provided by the reporter will fix the problem by setting white as the background colour of the page in addition to the background image.

Changed 3 years ago by vipsoft

that sounds counter intuitive... if the user has changed the default background color in their browser config, isn't the right approach to respect that (even if it looks bad)?

Changed 3 years ago by feyp

Yes, you should respect the default settings of the browser. But: If you set a foreground colour (i.e. text colour) you need to set the background colour, too.

Suppose one likes white text on black background. While not everyone's taste, you can still read it. So if a web page does not set either background colour or foreground colour, you're fine with this default setting. Now consider this: A web page does not set background colour, but sets text colour to black. You end up with black text on black background. The result: Unreadable.

Piwik.org sets a foreground colour. That's why it should set a sensible background-colour, too.

Changed 3 years ago by black silence

Comment 6 by feyp is absolutely right. If you define one colour (text or background) you have to define the other too or bad things may happen, see the screenshot for an example.

This is a common problem all over the internet, but only few people (like me) notice it.

Changed 3 years ago by matt

  • status changed from reopened to closed
  • resolution set to fixed
  • milestone set to RobotRock

Changed 3 years ago by black silence

nice, thanks very much

Changed 3 years ago by alivenk

Note: See TracTickets for help on using tickets.