Closed
Bug 702144
Opened 13 years ago
Closed 13 years ago
Scratchpad russian codepage
Categories
(DevTools :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 12
People
(Reporter: northtech, Assigned: dchanm+bugzilla)
References
Details
(Keywords: qawanted)
Attachments
(4 files, 2 obsolete files)
User Agent: Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20100101 Firefox/9.0 Build ID: 20111109112850 Steps to reproduce: Scratchpad - File - Open File - naroddisknocaptcha@script.user.js (utf-8) - see screen-shot (76867.png) If i write in clear Scratchpad - all normal.
Updated•13 years ago
|
Version: unspecified → 9 Branch
Updated•13 years ago
|
Attachment #574178 -
Attachment mime type: application/octet-stream → text/javascript
Comment 2•13 years ago
|
||
Thanks for the bug report! This looks like it works fine in Firefox 10+ with Orion enabled. Presumably this will be a problem in Firefoxes 8-10 with the text-based source editor in place.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 4•13 years ago
|
||
Strange. I'm unable to reproduce this with your attached text on my machine in OS X. Can anyone with a Windows machine try this out?
Keywords: qawanted
Assignee | ||
Comment 5•13 years ago
|
||
Not all the characters are displaying for me either. Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0a1) Gecko/20111108 Firefox/10.0a1 ID:20111108031146 OS X 10.6.8 Copying and pasting works fine. The script file also displays properly in Dashcode
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → dchan+bugzilla
Assignee | ||
Comment 6•13 years ago
|
||
I copied the code from exportToFile and it appears to work. We may be able to remove the manual conversion once bug 655658 lands.
Attachment #574738 -
Flags: feedback?
Assignee | ||
Comment 7•13 years ago
|
||
We probably want to pass the unicode version of the content to the callback. Stupid mistake
Attachment #574738 -
Attachment is obsolete: true
Attachment #574738 -
Flags: feedback?
Attachment #574743 -
Flags: feedback?
Will be implemented by the wrap of long lines in scratchpad (style editor and other) and file://, ftp://, http://, https://, etc protocols (in default or checkbox in settings)?
Comment 9•13 years ago
|
||
Comment on attachment 574743 [details] [diff] [review] convert input to UTF-8 v2 this looks like it should do the trick.
Attachment #574743 -
Flags: feedback? → feedback+
Assignee | ||
Comment 10•13 years ago
|
||
Hmmm, looking at the documentation again. I'm not sure if converter.Finish() needs to be called when converting to unicode. It seems to work fine without it. Does anyone have more experience with nsIScriptableUnicodeConverter?
Assignee | ||
Comment 11•13 years ago
|
||
Remove the Finish() call since it is only needed for ConvertFromUnicode() and not ConvertToUnicode() I need help landing the patch on m-c
Attachment #574743 -
Attachment is obsolete: true
Attachment #581829 -
Flags: review?(rcampbell)
Comment 13•13 years ago
|
||
Comment on attachment 581829 [details] [diff] [review] convert input to UTF-8 v3 I thought I reviewed this already. Sorry for the delay.
Attachment #581829 -
Flags: review?(rcampbell) → review+
Updated•13 years ago
|
Whiteboard: [land-in-fx-team]
Comment 14•13 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/805facc7eb21
Status: NEW → ASSIGNED
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Comment 15•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/805facc7eb21
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 12
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•