unusable with inverse theme

[c7d2f5c8667d26fffd5e7772d632c76d]

Steps to reproduce:
0) Change theme to "High contrast inverse"
1) Log out
2) Log in
3) Wait until the notification popup appears

Result:
The notification popup is not using high contrast inverse; its background is
white, and the text is unreadable (white too?) .
See attached screenshot.

[c7d2f5c8667d26fffd5e7772d632c76d (c7d2f5c8667d26fffd5e7772d632c76d-deactivatedaccount)]

Created an attachment (id=4900)
screenshot showing the problem

[Michael Vogt (mvo)]

This is fixed in the latest version of the notification-daemon. Please reopen if it is still a problem for you.

Cheers,
 Michael

[c7d2f5c8667d26fffd5e7772d632c76d (c7d2f5c8667d26fffd5e7772d632c76d-deactivatedaccount)]

Using notification-daemon 0.3.4-0u3, the notification popup is still using the same colours as in non-inverse themes, black-on-yellow aka. unreadable if you need the inverse theme.

I guess this bug should be re-assigned to notification-daemon then?

[Michael Vogt (mvo)]

Thanks, I can reproduce this now.

[Sebastian Dröge (slomo)]

bug #34434 (marked as dup of this one) had a rather hacky patch which could definitely be improved:
http://librarian.launchpad.net/1735638/hardcode_text_colour.patch

[Michael Vogt (mvo)]

patch that uses the bg of the tooltips but this changes the color of the notification

[Michael Vogt (mvo)]

current notification color

[Michael Vogt (mvo)]

new notificaton color based on tooltips

[Michael Vogt (mvo)]

The attached patch solves the problem by using the tooltip color for the notification. This means that under a inverse theme we get a inverse notification (as expected).

The problem is that it changes the color of the current notitifcations in the human theme to a stronger yellow than what we ship currently.

I will ask for feedback from our art people if that is a change we can do for dapper.

Thanks,
 Michael

[Michael Vogt (mvo)]

Fixed with the upload of notification-daemon_0.3.4-0ubuntu7. The background is based on the color of the tooltip now.