Ubuntu
notification-daemon package

notify bubble has text across screens

Bug #31433 reported by aent
36
Affects Status Importance Assigned to Milestone
notification-daemon
New
Unknown
notification-daemon (Ubuntu)
Fix Released
Medium
Michael Vogt

Bug Description

When you position the notification area on the left side of the right monitor when you have multiple monitors, the notification bubble will start with the text's left side directly under the icon, and the bubble will thus expand onto the second monitor. This causes the text inside the bubble to be displayed between the screens, making it very difficult to read.

It should act like normal tooltips, the calendar when clicking on the clock, etc. so that it will display itself entirely on one screen.

Sebastien Bacher (seb128)
Changed in notification-daemon:
assignee: nobody → mvo
Revision history for this message
Martijn vdS (martijn) wrote :

The same happens if you place it on the right side of your left screen.

Changed in notification-daemon:
status: Unconfirmed → Confirmed
Revision history for this message
Johan Kiviniemi (ion) wrote : Xinerama support for src/daemon.c

This patch should fix the problem. I looked at http://trac.galago-project.org/attachment/ticket/5/notification-daemon_xinerama.patch as an example, but this patch uses the first monitor instead of the rightmost monitor.

It would be best to allow the user to configure which monitor to show the notifications on, but the first monitor is a good default for now because it's likely to be the primary one.

For example i have a TV connected to the second head, and it's not turned on all the time. All the arrowless notifications are opened on it currently.

Revision history for this message
Johan Kiviniemi (ion) wrote : debian/patches/03_xinerama.patch

The previous patch wasn't enough after all. This *should* fix the problem, but still needs testing.

This file patches src/daemon.c and the upstream default theme. The Ubuntu theme is added in debian/patches/01_ubuntu_theme.diff. I'm going to upload an updated version of it after this.

Revision history for this message
Johan Kiviniemi (ion) wrote : Updated debian/patches/01_ubuntu_theme.diff

See above.

Revision history for this message
Johan Kiviniemi (ion) wrote :

A version of notification-daemon with those patches added is available at:

## wget -q -O- http://johan.kiviniemi.name/pubkey | sudo apt-key add -
deb http://johan.kiviniemi.name/ubuntu/ dapper notification-daemon
deb-src http://johan.kiviniemi.name/ubuntu/ dapper notification-daemon

Please test it and report whether it works or not.

Revision history for this message
Johan Kiviniemi (ion) wrote : 01_ubuntu_theme.patch

I found one bug from the patch with libnotify/tests/test-xy-stress. It should be fixed now.

Revision history for this message
Johan Kiviniemi (ion) wrote : 03_xinerama.patch

Ditto.

Revision history for this message
Michael Vogt (mvo) wrote :

Thanks for your patch, I applied it into my local tree and prepare a new upload now.

Changed in notification-daemon:
status: Confirmed → Fix Committed
Revision history for this message
Michael Vogt (mvo) wrote :

Fixed with the upload of notification-daemon_0.3.4-0ubuntu6

Changed in notification-daemon:
status: Fix Committed → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in notification-daemon:
status: Unknown → New
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.