The default bug view has changed. See this FAQ.

Mojibake in context menus on the chat conversation

RESOLVED FIXED in Thunderbird 14.0

Status

Thunderbird
Instant Messaging
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: marsf, Assigned: florian)

Tracking

({jp-critical})

13 Branch
Thunderbird 14.0
x86
Windows 7
jp-critical

Thunderbird Tracking Flags

(thunderbird13 fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 617314 [details]
mojibake in chat context menu

User Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; ja; rv:1.9.2.28) Gecko/20120306 Firefox/3.6.28
Build ID: 20120306064154

Steps to reproduce:

1. Open chat tab.
2. Connect twitter account.
3. Right click individual tweet to show its context menu.


Actual results:

Only the menus of twitter actions (retweet, reply, follow, ... ) has mojibake.
It seems that the localized UTF-8 strings are not properly handled.
(Reporter)

Updated

5 years ago
Keywords: jp-critical
(Assignee)

Comment 1

5 years ago
I'm a bit confused by this bug report, as:
- I've tried to replace the en-US strings with strings containing UTF8 characters in my debug build, and these strings were displayed correctly.
- I've tried to find the ja translation of these strings, but http://hg.mozilla.org/l10n-central/ja/file/tip/chat/twitter.properties doesn't exist (I've checked the URL with the French locale, and http://hg.mozilla.org/l10n-central/fr/file/tip/chat/twitter.properties#l20 does exist).
(Reporter)

Comment 2

5 years ago
I've tested on Aurora build. (We haven't updated l10n-central.)
Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20120422 Thunderbird/13.0a2

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ja/file/47eeb04b0b3c/chat/twitter.properties

Tooltips and options are displayed correctly.
(Assignee)

Comment 3

5 years ago
Created attachment 617456 [details] [diff] [review]
Fix

I couldn't reproduce with a string containing French non ASCII characters, but I can reproduce using the strings in the ja localized file.

It's caused by a trivial type error in the idl interface.

Thanks for reporting this!
Assignee: nobody → florian
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #617456 - Flags: review?(clokep)
Attachment #617456 - Flags: approval-comm-aurora?
Comment on attachment 617456 [details] [diff] [review]
Fix

This change looks fine, thanks for fixing it Florian.
Attachment #617456 - Flags: review?(clokep) → review+
(Assignee)

Comment 5

5 years ago
http://hg.mozilla.org/comm-central/rev/9869097a829e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0

Updated

5 years ago
Attachment #617456 - Flags: approval-comm-aurora? → approval-comm-aurora+
Drive-by comment here - should the UUID for that IDL have been changed?

Comment 7

5 years ago
(In reply to Mike Conley (:mconley) from comment #6)
> Drive-by comment here - should the UUID for that IDL have been changed?

I don't think Thunderbird cares since we haven't turned this on, but Instantbird might...
(Assignee)

Comment 8

5 years ago
(In reply to David :Bienvenu from comment #7)
> (In reply to Mike Conley (:mconley) from comment #6)
> > Drive-by comment here - should the UUID for that IDL have been changed?
> 
> I don't think Thunderbird cares since we haven't turned this on, but
> Instantbird might...

I've never applied that rule for Instantbird as I'm not aware of any binary add-on using our xpcom interfaces.
Looks like this was pushed to aurora earlier today:

http://hg.mozilla.org/releases/comm-aurora/rev/88ca33153d6a
status-thunderbird13: --- → fixed
You need to log in before you can comment on or make changes to this bug.