Closed
Bug 155269
Opened 24 years ago
Closed 24 years ago
Categories
(MailNews Core :: Internationalization, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: naving, Assigned: naving)
Details
(Whiteboard: [ish1+][fixedish1])
Attachments
(1 file)
|
4.38 KB,
patch
|
naving
:
review+
naving
:
superreview+
|
Details | Diff | Splinter Review |
filing this bug because the fix is in mozilla code. comments from bugscape bug.
Opened: 2002-06-14 18:08
Build: N7.0PR1 4th zh-cn build
When sending an email, the folder name "sent" is in English in the pop-up dialog.
------- Additional Comment #1 From Yuying Long 2002-06-14 18:10 -------
Created an attachment (id=6976)
screen shot for detail
------- Additional Comment #2 From Yuying Long 2002-06-17 13:34 -------
Created an attachment (id=7006)
screen shot for "Drafr" folder
------- Additional Comment #3 From ji@netscape.com 2002-06-17 13:54 -------
Bug 15585 was filed for the problem in Ja build. It could be a l12y problem.
------- Additional Comment #4 From Naoki Hotta 2002-06-17 17:06 -------
cc to naving@netscape.com
Naving, can the patch of http://bugzilla.mozilla.org/show_bug.cgi?id=151859 will
this also?
------- Additional Comment #5 From rchen@netscape.com 2002-07-01 13:58 -------
It seems the core bug is still there. We localized messenger.properties in
06/21 build but still see the problem. Yuying, can you follow up? We may need to
reopen the bugzilla 151859.
------- Additional Comment #6 From Navin Gupta 2002-07-01 14:04 -------
I never tried to fix this bug. I mean the fix in bugzilla 151859 won't fix this.
------- Additional Comment #7 From Navin Gupta 2002-07-01 15:12 -------
taking, fix upcoming.
------- Additional Comment #8 From Navin Gupta 2002-07-01 15:26 -------
Created an attachment (id=7353)
proposed fix
The fix is to use the folder name instead of parsing the uri to get the name
(which is wrong).
------- Additional Comment #9 From Navin Gupta 2002-07-01 15:30 -------
JF, David Can I get reviews for upcoming patch ? thx. I removed
GetFolderNameFromURLString() because it is not used anywhere else.
------- Additional Comment #10 From Navin Gupta 2002-07-01 15:30 -------
Created an attachment (id=7354)
proposed fix
The fix is to use the folder name instead of parsing the uri to get the name
(which is wrong).
------- Additional Comment #11 From Navin Gupta 2002-07-01 15:31 -------
(From update of attachment 7353 [details])
wrong one
------- Additional Comment #12 From Navin Gupta 2002-07-01 15:32 -------
Created an attachment (id=7355)
proposed fix
Sorry, this is the right one.
The fix is to use the folder name instead of parsing the uri to get the name
(which is wrong).
------- Additional Comment #13 From Jean-Francois Ducarroz 2002-07-01 15:41 -------
(From update of attachment 7355 [details])
looks good. R=ducarroz
------- Additional Comment #14 From bienvenu@netscape.com 2002-07-01 15:57 -------
(From update of attachment 7355 [details])
sr=bienvenu
| Assignee | ||
Comment 1•24 years ago
|
||
| Assignee | ||
Updated•24 years ago
|
Attachment #89846 -
Flags: superreview+
Attachment #89846 -
Flags: review+
| Assignee | ||
Comment 2•24 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 3•24 years ago
|
||
nsbeta1 - according to Ray Chen's comment in bugscape bug report:
Can we check in to branch? nominate "nsbeta1".
Keywords: nsbeta1
Navin, is the fix checked into the branch? And if "yes", Ray or Ying, did the
localized build picked it up so i can verify it?
| Assignee | ||
Comment 5•24 years ago
|
||
It was never checked on branch.
Comment 6•23 years ago
|
||
can we check it into branch?
Comment 7•23 years ago
|
||
all buffy work is being done on the trunk right now.
We definitely need this for the branch.
This is verified on the trunk, please see
http://bugscape.mcom.com/show_bug.cgi?id=16810#c20
Whiteboard: [ish]
Updated•23 years ago
|
Whiteboard: [ish] → [ish1+]
Updated•21 years ago
|
Product: MailNews → Core
Updated•18 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•