Closed
Bug 403435
Opened 18 years ago
Closed 18 years ago
Change default setting for standard URL encoding for Korean
Categories
(Mozilla Localizations :: ko / Korean, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: channy, Assigned: channy)
Details
Attachments
(2 files)
|
36.32 KB,
image/gif
|
Details | |
|
302 bytes,
patch
|
Pike
:
approval1.8.1.12-
|
Details | Diff | Splinter Review |
Firefox's default setting of URL encoding escaped Korean characters, so korean users are hard to recognize permanent URL including Korean characters. By changing some setting, users can see korean characters in address bar.
| Assignee | ||
Comment 1•18 years ago
|
||
Attachment #288269 -
Flags: approval1.9?
Attachment #288269 -
Flags: approval1.8.1.11?
Comment 2•18 years ago
|
||
Comment on attachment 288269 [details] [diff] [review]
firefox-l10n.js patch
This change doesn't seem to do what you want, at least it doesn't for both me and smontagu.
Note, on the trunk, this "just works", so I'm inclined to WORKSFORME this bug and not risk anything on the branch.
Attachment #288269 -
Flags: approval1.9?
Attachment #288269 -
Flags: approval1.8.1.11?
Attachment #288269 -
Flags: approval1.8.1.11-
Comment 3•18 years ago
|
||
This isn't a l10n bug -- I don't think any locale ever changed a network pref. I wouldn't accept such a patch. Also, I don't think this is something for the 1.8.1 branch. On trunk we're already unescaping addresses.
Comment 4•18 years ago
|
||
So this has already been checked in on trunk?
http://mxr.mozilla.org/l10n/source/ko/browser/firefox-l10n.js
| Assignee | ||
Comment 5•18 years ago
|
||
(In reply to comment #4)
> So this has already been checked in on trunk?
> http://mxr.mozilla.org/l10n/source/ko/browser/firefox-l10n.js
>
It was just test checkin for escaping in nightly test of trunk. These setting might not be not included in beta 1 build. I'll regress this check-in.
I confirmed your comment in trunk. Is it included in 1.8.1 branch?
Comment 6•18 years ago
|
||
(In reply to comment #5)
> I confirmed your comment in trunk. Is it included in 1.8.1 branch?
No, I'm still working on regressions. This is nothing for stable branches.
| Assignee | ||
Comment 7•18 years ago
|
||
(In reply to comment #6)
> (In reply to comment #5)
> > I confirmed your comment in trunk. Is it included in 1.8.1 branch?
>
> No, I'm still working on regressions. This is nothing for stable branches.
>
Would you let me know bug number?
Comment 8•18 years ago
|
||
| Assignee | ||
Comment 9•18 years ago
|
||
It's fixed in Firefox 3 setting.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•