| Summary: | notification popups do not support UTF8 | ||
|---|---|---|---|
| Product: | TDE | Reporter: | Francois Andriot <albator78> |
| Component: | tdelibs | Assignee: | Slávek Banko <slavek.banko> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | albator78, bugwatch, darrella, slavek.banko |
| Priority: | P5 | ||
| Version: | 3.5.13 [Trinity] | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Compiler Version: | TDE Version String: | ||
| Application Version: | Application Name: | ||
| Attachments: |
kdbusnotification : fix utf8 messages
Fix missing DBUS version macros |
||
|
Description
Francois Andriot
2013-02-17 13:47:38 CST
UTF8 works with kdialog: kdialog --title "Bébé" --msgbox "bébé bébé bébé" Is the problem actually zenity or GTK? Using the same zenity command under KDE4 displays the popup normally ... Also, any popup message displayed under TDE has the problem. For example, when I unplug the AC adapter from my laptop, kpowersave shows a garbage message too ... Mozilla thunderbird too ... Zenity is just a way to reproduce the problem. It's not specific to the application that sends the message. Francois, Tim, How much work is required to fix this? Should we add this report to the R14.0.0 etherpad? It looks like the issue is limited to some GTK2 applications after all. E.G: - when using TDE's krandrtray to change resolution, the text in popup appears correctly with french caracters. - I cannot reproduce the bug with Zenity (do not understand why, it was so easy) - Firefox and Thunderbird still show badly encoded messages, whereas the same messages are correct under KDE4. Created attachment 1352 [details]
kdbusnotification : fix utf8 messages
OK I've found that the messages that were badly displayed were coming through kdbusnotification.
The attached patch fixes this.
Now everything looks OK.
I was going to test this patch when I realized the patch is for kdbusnotification. I never built or installed this package. The sources contain no notes or README files explaining the purpose of the package or for whom the package is intended. I can't find any references on the web. So, um, my silly questions are what does this package do and for whom is the package intended? Hello, this package contains the "notification daemon" for TDE, using "libnotify". It is a standard way to allow applications to display popup notifications in any Desktop environment (KDE4, gnome ...). For example, Firefox send a message "All download finished", Thunderbird sends message "you have 3 new mails". When kdbusnotification is installed in TDE, the messages are displayed as small popups at the bottom-right of the screen, near the clock. If kdbusnotification is not installed, you never see these messages. If you don't know what the messages look like, they look the same as some TDE built-in popups, for example: - when yakuake starts - when tderandrtray has changed the resolution - when tdenetworkmanager has connected to a network This utility is very convenient, I never use TDE without it. Thank you for the information. I learn something new every day. :-) I will try building the package along with the proposed patch. Not related to the patch but I see the following in my build log: daemon.cpp:162:30: warning: "DBUS_MAJOR_VER" is not defined [-Wundef] daemon.cpp:162:30: warning: "DBUS_MINOR_VER" is not defined [-Wundef] I built the package along with the proposed patch. I see notification-daemon-tde in my process list. So far so good --- is there a simple way to test that kdbusnotification is functioning? > is there a simple way to test that kdbusnotification is functioning?
Never mind. I found notify-send on my system. The popups work fine.
Without the patch, when I run notify-send "bébé" I see garbage for é.
With the patch the popup displays as expected.
Francois, do you have permissions to push to git?
Pushed to GIT in hash 3542ae76. Thank you all for your work. What about the build errors mentioned in comment 9? (Odpověď na komentář #13)
> What about the build errors mentioned in comment 9?
Good point - this is not yet resolved!
Created attachment 1430 [details]
Fix missing DBUS version macros
Try attached patch...
The patch eliminated the build warnings and the package functions as expected, with UTF-8 working too. Good job! Pushed to GIT in hash 1b6c7812. |