Closed
Bug 234466
Opened 17 years ago
Closed 12 years ago
Syntax highlighting is broke for view source of mail messages (should be disabled)
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b3
People
(Reporter: raccettura, Assigned: philor)
References
Details
Attachments
(1 file)
2.11 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
toggle the menu option results in the following: Error: uncaught exception: [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIWebPageDescriptor.currentDescriptor]" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: chrome://global/content/viewSource.js :: highlightSyntax :: line 539" data: no] Not sure what's going on here. Tried to take care of this myself, but with no success. Any hints?
Comment 1•17 years ago
|
||
Syntax highlighting doesn't work for mail messages in thunderbird or seamonkey.
Reporter | ||
Comment 2•17 years ago
|
||
Hmm. I haven't tried seamonkey since 1.5. (1.6 nightly's installer is broken for me, making it more of a pain). Played with this the other night. Still can't figure it out.
Viewer syntax highlighting doesn't work for mail messages in thunderbird 1.0+. Thunderbird 1.0+ / 1.8b4: 2005.07.19.04 - windows
Comment 4•16 years ago
|
||
Steps to reproduce?
Steps to reproduce ? Same as Feb 2004: just try it ! 1. Select a html message 2. > view > message source 3. > view > Syntax highlight Thunderbird 1.0+ / 1.8b4: 2005.07.21.04 - windows
Bug still confirmed with Thunderbird (Windows NT 5.0; rv:1.8b4) Gecko/20050912 »» Page source viewer syntax highlighting works fine on Firefox 1.5B1
still confirmed; bug closed or forgotten ? _________________________ ThunderBird version 2 alpha 1 (2006.03.09)
Comment 8•15 years ago
|
||
as much as diorser loves this bug, we aren't going to block the release on this. That being said, adding the helpwanted keyword, feel free to jump in and hack on it.
Flags: blocking-thunderbird2? → blocking-thunderbird2-
Keywords: helpwanted
Assignee | ||
Updated•14 years ago
|
QA Contact: front-end
Updated•13 years ago
|
Assignee: mscott → nobody
Comment 10•12 years ago
|
||
Did this option ever worked in TB ? Any (new) ideas why it doesn't ?
Assignee | ||
Comment 11•12 years ago
|
||
There are two problems: * we get the error (now an NS_ERROR_NOT_AVAILABLE error instead of comment 0's NS_ERROR_FAILURE), because the toggle just changes the pref and calls PageLoader.loadPage(PageLoader.currentDescriptor... to avoid hitting the network, but because we have disablehistory on the view source browser, .currentDescriptor fails * it will never work, since highlight_syntax only works when what you are viewing is the source of an HTML document as parsed by nsViewSourceHTML.cpp both of which are neatly "solved" by disabling the menuitem.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #375498 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Updated•12 years ago
|
Comment 12•12 years ago
|
||
Are You saying that when a message is content: text/html, that the HTML parsing by nsViewSourceHTML.cpp does not take place in Tb, thus the highlighter is doomed to fail?
Assignee | ||
Comment 13•12 years ago
|
||
I'm saying that when you View Message Source the document that is loaded in the <browser> is always a message, never text/html, whether or not the only body in the message is text/html.
Comment 14•12 years ago
|
||
(In reply to comment #11) > * it will never work, since highlight_syntax only works when what you are > viewing is the source of an HTML document as parsed by nsViewSourceHTML.cpp [...] > both of which are neatly "solved" by disabling the menuitem. hmm so another bug would be needed to provide modified ver of syntax highlighting for TB
Assignee | ||
Comment 15•12 years ago
|
||
Another bug which would need an hero to fix it, and would be at great risk of being wontfixed when someone came face to face with the patch, since it would very probably involve forking viewsource to have our own copy that knows that load/reload involves telling something to tell libmime to break apart a message and pass this part through this, and that part through that, and the other part through the other thing and then reassemble the parts into a single document that would pretend to be the message source.
Updated•12 years ago
|
Summary: Syntax highlighting is broke for view source → Syntax highlighting is broke for view source of mail messages (should be disabled)
Comment 16•12 years ago
|
||
Comment on attachment 375498 [details] [diff] [review] Fix Sure, r=mkmelin
Attachment #375498 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 17•12 years ago
|
||
http://hg.mozilla.org/comm-central/rev/bf8d5258553b
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 18•12 years ago
|
||
(In reply to comment #11) > both of which are neatly "solved" by disabling the menuitem. Why don't you *hide* the useless menu item instead of just disabling it?
You need to log in
before you can comment on or make changes to this bug.
Description
•