Closed
Bug 716481
Opened 13 years ago
Closed 13 years ago
Spellcheck icon stays disabled if clicking on a mailto link with a subject parameter, or replying (any action where the cursor starts in the body field)
Categories
(Thunderbird :: Message Compose Window, defect)
Tracking
(thunderbird10- affected, thunderbird11+ fixed, thunderbird12 fixed, thunderbird-esr1011+ fixed)
RESOLVED
FIXED
Thunderbird 13.0
People
(Reporter: psm, Assigned: standard8)
References
()
Details
(Keywords: regression, Whiteboard: [gs])
Attachments
(2 files)
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
iannbugzilla
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:9.0.1) Gecko/20100101 Firefox/9.0.1 Build ID: 20111220165912 Steps to reproduce: Clicking on a mailto-link with a subject parameter (with or without body parameter). See attached html-file for examples of such links. Actual results: A Thunderbird message compose window opens with the cursor already located in the mail body area. The spell check icon in the toolbar stays disabled (greyed out). Expected results: The spell check icon should be enabled. Like when you click on a normal mailto link (without parameters) and then manually move the cursor to the mail body area.
Reporter | ||
Comment 1•13 years ago
|
||
Reproduced on Windows XP, Windows 7 and Ubuntu Linux. Bug is present in Version 9.01 and 10.0b1 but not in Thunderbird 8.0 Workaround: Click on subject or recipient fields and then click back to the mail body area.
OS: Windows XP → All
Hardware: x86 → All
Comment 2•13 years ago
|
||
(In reply to Michael Fischer from comment #1) Works for me. The spell check button is not greyed out. I tried the mailto link in your attachment having subject and body. User Agent: Mozilla/5.0 (X11; Linux i686; rv:9.0) Gecko/20111222 Thunderbird/9.0.1 Application Build ID: 20111222141531 Try it without addons (start Thunderbird in safe-mode) and check.
I am having a similar problem now with 9.0.1, as are others in our office, but it has nothing to do with mailto links... If I write a *new* message, spellcheck works as expected. However, if I *Reply* to any message, the Spellcheck button is disabled/greyed out, unless/until I SHIFT+TAB then TAB to move the cursor from the body to the subject and then back to the body. This re-enables the spellcheck button/functionality every time.
Incidentally, we also use spellcheck on Send, and that is working (at least for me), even though the button is greyed out (so cannot do a manual spellcheck).
Reporter | ||
Comment 5•13 years ago
|
||
Forgot to say that I tried in safe-mode and User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20111222 Thunderbird/9.0.1 I can also confirm Charles issue with the reply button. Spellcheck on Send works of course but you can't select the language as long as the button is grayed out. Also my coworkers find it annoying and prefer only "mark as you type".
Reporter | ||
Comment 7•13 years ago
|
||
(In reply to Ludovic Hirlimann [:Usul] from comment #6) > Can you guys hunt when this regressed ? You mean find out in which version this error first occurred? In 8.0 it worked fine, in 9.0.1 it was broken. If you could point me to a server where I can download the intermediate versions, I could try to find it out.
(In reply to Charles from comment #3) > I am having a similar problem now with 9.0.1, as are others in our office, > but it has nothing to do with mailto links... > > If I write a *new* message, spellcheck works as expected. > > However, if I *Reply* to any message, the Spellcheck button is > disabled/greyed out, unless/until I SHIFT+TAB then TAB to move the cursor > from the body to the subject and then back to the body. This re-enables the > spellcheck button/functionality every time. Same here: When replying to first e-mail after start of TB (by clicking on the "reply" button), the spell check button in the opening message composition window is grayed out. The spell check works at the same time - i.e. the language one used earlier is still checked, but the problem is that one cannot change the language, because the button is grayed out. As mentioned in previous comments, the workaround is changing the cursor to the subject or recipient field, and back to the main message field. Also, the problem does not appear on the second message one replies to later. It seems that the problem does not appear on when a new message is composed (instead of replying), because, when replying, the cursor is automatically positioned in the main text field of the composition window, as opposed to the "recipient" field when a new message is composed. I'm not 100 percent sure, but I think it came in a version before 9. Maybe in 8 or so. One of the later ones, anyway. ---------------------------- TB information (German version): Allgemeine Informationen Name: Thunderbird Version: 9.0.1 User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20111222 Thunderbird/9.0.1 Profilordner: Beinhaltenden Ordner anzeigen (Lokaler DatentrΓ€ger) Build-ID der Anwendung: 20111222160251 Aktivierte Plugins: about:plugins Build-Konfiguration: about:buildconfig Absturzberichte: about:crashes E-Mail- und Newsgruppen-Konten account1: INCOMING: account1, , (imap) imap.gmx.net:993, SSL, passwordCleartext OUTGOING: mail.gmx.net:465, SSL, passwordCleartext, true OUTGOING: mail.gmx.net:465, SSL, passwordCleartext, false OUTGOING: mail.gmx.net:465, SSL, passwordCleartext, false OUTGOING: mail.gmx.net:465, SSL, passwordCleartext, false OUTGOING: mail.gmx.net:465, SSL, passwordCleartext, false OUTGOING: mail.gmx.net:465, SSL, passwordCleartext, false account2: INCOMING: account2, , (none) Local Folders, plain, passwordCleartext account3: INCOMING: account3, , (rss) Feeds, plain, passwordCleartext Erweiterungen Highlighter, 0.6.5, true, andre.rodier-highlighter@gmailcom Russian spellchecking dictionary, 0.4.4, true, ru@dictionaries.addons.mozilla.org Test Pilot for Thunderbird, 1.3.4, true, tbtestpilot@labs.mozilla.com United States English Spellchecker, 5.0.1, true, en-US@dictionaries.addons.mozilla.org WΓΆrterbuch Deutsch (de-DE), Hunspell-unterstΓΌtzt, 20111003, true, de_DE@dicts.j3e.de Modifizierte Einstellungen Name: Wert accessibility.typeaheadfind.flashBar: 0 extensions.lastAppVersion: 9.0.1 gfx.blacklist.direct2d: 2 gfx.blacklist.layers.direct3d10: 2 gfx.blacklist.layers.direct3d10-1: 2 mail.openMessageBehavior.version: 1 network.cookie.cookieBehavior: 2 network.cookie.prefsMigrated: true places.database.lastMaintenance: 1326727977 places.history.expiration.transient_current_max_pages: 26585 places.history.expiration.transient_optimal_database_size: 42534666 print.print_printer: PDFCreator print.printer_PDFCreator.print_bgcolor: false print.printer_PDFCreator.print_bgimages: false print.printer_PDFCreator.print_command: print.printer_PDFCreator.print_downloadfonts: false print.printer_PDFCreator.print_edge_bottom: 0 print.printer_PDFCreator.print_edge_left: 0 print.printer_PDFCreator.print_edge_right: 0 print.printer_PDFCreator.print_edge_top: 0 print.printer_PDFCreator.print_evenpages: true print.printer_PDFCreator.print_footercenter: print.printer_PDFCreator.print_footerleft: &PT print.printer_PDFCreator.print_footerright: &D print.printer_PDFCreator.print_headercenter: print.printer_PDFCreator.print_headerleft: &T print.printer_PDFCreator.print_headerright: &U print.printer_PDFCreator.print_in_color: true print.printer_PDFCreator.print_margin_bottom: 0.393750011920929 print.printer_PDFCreator.print_margin_left: 0.5 print.printer_PDFCreator.print_margin_right: 0.5 print.printer_PDFCreator.print_margin_top: 0.393750011920929 print.printer_PDFCreator.print_oddpages: true print.printer_PDFCreator.print_orientation: 0 print.printer_PDFCreator.print_page_delay: 50 print.printer_PDFCreator.print_pagedelay: 500 print.printer_PDFCreator.print_paper_data: 9 print.printer_PDFCreator.print_paper_height: 11,00 print.printer_PDFCreator.print_paper_size_type: 0 print.printer_PDFCreator.print_paper_size_unit: 1 print.printer_PDFCreator.print_paper_width: 8,50 print.printer_PDFCreator.print_reversed: false print.printer_PDFCreator.print_scaling: 1,00 print.printer_PDFCreator.print_shrink_to_fit: true print.printer_PDFCreator.print_to_file: false print.printer_PDFCreator.print_unwriteable_margin_bottom: 0 print.printer_PDFCreator.print_unwriteable_margin_left: 0 print.printer_PDFCreator.print_unwriteable_margin_right: 0 print.printer_PDFCreator.print_unwriteable_margin_top: 0 security.OCSP.require: true Grafik Karten-Beschreibung: Intel(R) Graphics Media Accelerator 3150 Vendor-ID: 8086 GerΓ€te-ID: a011 Karten-RAM: Unknown Karten-Treiber: igxprd32 Treiber-Version: 6.14.10.5134 Treiber-Datum: 9-24-2009 Vendor-ID (GPU #2): 8086 GerΓ€te-ID (GPU #2): a012 Karten-RAM (GPU #2): Unknown Karten-Treiber (GPU #2): Unknown Treiber-Version (GPU #2): 6.14.10.5134 Treiber-Datum (GPU #2): 9-24-2009 WebGL-Renderer: Wurde auf Grund Ihrer Grafiktreiberversion blockiert. Versuchen Sie, Ihren Grafiktreiber auf mindestens Version 6.14.10.5260 zu aktualisieren. GPU-beschleunigte Fenster: 0/1. Wurde auf Grund Ihrer Grafiktreiberversion blockiert. Versuchen Sie, Ihren Grafiktreiber auf mindestens Version 6.14.10.5260 zu aktualisieren.
Assignee | ||
Updated•13 years ago
|
Status: UNCONFIRMED → NEW
tracking-thunderbird10:
--- → ?
tracking-thunderbird11:
--- → +
Ever confirmed: true
Summary: Spellcheck icon stays disabled if clicking on a mailto link with a subject parameter → Spellcheck icon stays disabled if clicking on a mailto link with a subject parameter, or replying (any action where the cursor starts in the body field)
Comment 10•13 years ago
|
||
Uh-Oh I can reproduce in Mozilla/5.0 (Windows NT 5.0; rv:10.0) Gecko/20120118 Thunderbird/10.0
status-thunderbird10:
--- → affected
Comment 11•13 years ago
|
||
The "or replying (any action where the cursor starts in the body" symptom exist on: Mozilla/5.0 (Windows NT 5.0; rv:12.0a1) Gecko/20120120 Thunderbird/12.0a1 If you try to reply a second time, you run into bug 567240 ..but the spellcheck icon is no longer grayed out.
Assignee | ||
Comment 12•13 years ago
|
||
(In reply to Joe Sabash from comment #10) > Uh-Oh I can reproduce in > Mozilla/5.0 (Windows NT 5.0; rv:10.0) Gecko/20120118 Thunderbird/10.0 Seeing as this was reported against TB 9, that's not surprising.
Comment 13•13 years ago
|
||
I have a "feeling"(have not checked) that this bug has been around for a while, maybe since bug 178324 fix was checked in.(http://hg.mozilla.org/mozilla-central/rev/cabb8925dcd3) Maybe some of the unfixed dependent bugs there could be a clue. Bug 567240 which has a similar workaround, established that regression range.
Assignee | ||
Updated•13 years ago
|
Comment 14•13 years ago
|
||
(In reply to Michael Fischer from comment #7) > (In reply to Ludovic Hirlimann [:Usul] from comment #6) > > Can you guys hunt when this regressed ? > > You mean find out in which version this error first occurred? > In 8.0 it worked fine, in 9.0.1 it was broken. If you could point me to a > server where I can download the intermediate versions, I could try to find > it out. http://ftp.mozilla.org/pub/mozilla.org/thunderbird/nightly/ using the comm-central builds.
Reporter | ||
Comment 15•13 years ago
|
||
Bug first occurs on the nightly builds of 20. August 2011: http://ftp.mozilla.org/pub/mozilla.org/thunderbird/nightly/2011/08/2011-08-20-03-00-14-comm-central/ Build of 19. August does not have this issue.
Comment 16•13 years ago
|
||
Thanks for the regression testing Michael. This bug is also present in: Mozilla/5.0 (Windows NT 5.0; rv:12.0a1) Gecko/20120125 Thunderbird/12.0a1 ID:20120125030312 I never noticed since I use "Check as you type" bug 581661 looks highly suspect https://hg.mozilla.org/comm-central/rev/8bbd7b60e5a8 Set blocking based on the regression date.
Blocks: 581661
Comment 17•13 years ago
|
||
I expect that most people use "Spellcheck as you type" But for those that use manual spellcheck, this bug makes it appear that a major feature is not available. This bug is present in Trunk, Branches and Beta Bumping severity. Adding cc's
Severity: normal → major
Comment 18•13 years ago
|
||
My two-cents' worth as a loyal Thunderbird user is that this bug could make users abandon Thunderbird and that it needs to be fixed with alacrity. Thanks.
Reporter | ||
Comment 19•13 years ago
|
||
(In reply to Joe Sabash from comment #17) > I expect that most people use "Spellcheck as you type" > But for those that use manual spellcheck, this bug makes it appear that a > major feature is not available. There's also the case of users that often change the language. I got noticed of the bug from our customer care employees that have to switch languages constantly (Switzerland with it's official four languages is a hard case for any spell-checker developer ;-)
Comment 20•13 years ago
|
||
Israel with its three official and 123 unofficial languages is even harder and yet all the spell-checkers worked fine until Thunderbird 9 came along.
Assignee | ||
Comment 21•13 years ago
|
||
Folks, I understand you may be frustrated, but please don't comment unless you are actively helping this bug move forward to being fixed. Please re-read the etiquette before commenting again: https://bugzilla.mozilla.org/page.cgi?id=etiquette.html - bugzilla is a place we work, and it needs to be a nice place to work without these sorts of comments. If you wish to discuss this, please email me privately. As regards to fixing the bug, we are actively tracking it for Thunderbird 11 (see the tracking-thunderbird11 flag), which means we'll be doing our best to fix it for that release.
Assignee | ||
Comment 23•13 years ago
|
||
This is actually a regression from bug 680511. It removed some code that is obviously needed, although I'm currently pondering if it is possible to put that code back in a slightly better/correct manner.
Assignee | ||
Comment 24•13 years ago
|
||
The issue is that we have disabled on the element on startup, and the code used to set it enabled. Bug 680511 removed the shared code but only the disabled attribute from the element for SeaMonkey, hence breaking Thunderbird. This patch seems to fix it (I was testing using reply to a newsgroup article, though there's other ways as well). The existing command handling seems to be good enough to ensure it is disabled at the correct times as well.
Assignee | ||
Comment 25•13 years ago
|
||
Try server builds should start appearing here shortly: https://ftp.mozilla.org/pub/mozilla.org/thunderbird/try-builds/bugzilla@standard8.plus.com-40b4b241f52a/
Comment 26•13 years ago
|
||
The above link does not seem valid. However, the spell-checker in TB11b2 seems to work.
Assignee | ||
Comment 27•13 years ago
|
||
(In reply to avner.falk@usa.net from comment #26) > The above link does not seem valid. I said shortly - i.e. in a little while, but not necessarily 3 minutes after I post the link. > However, the spell-checker in TB11b2 seems to work. Then you're not testing the exact problem listed in this bug.
Assignee | ||
Comment 28•13 years ago
|
||
(In reply to Mark Banner (:standard8) from comment #27) > (In reply to avner.falk@usa.net from comment #26) > > The above link does not seem valid. > > I said shortly - i.e. in a little while, but not necessarily 3 minutes after > I post the link. (apologies if this sounds harsh, I didn't mean it to).
Updated•13 years ago
|
Keywords: regression
Comment 29•13 years ago
|
||
Comment on attachment 599080 [details] [diff] [review] The fix r=me
Attachment #599080 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Comment 30•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/cf85b487f53c
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
Assignee | ||
Comment 31•13 years ago
|
||
Comment on attachment 599080 [details] [diff] [review] The fix [Triage Comment]
Attachment #599080 -
Flags: approval-comm-beta+
Attachment #599080 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 32•13 years ago
|
||
Checked in: http://hg.mozilla.org/releases/comm-aurora/rev/6f794e627928 http://hg.mozilla.org/releases/comm-beta/rev/b3fa8988e83b
status-thunderbird11:
--- → fixed
status-thunderbird12:
--- → fixed
tracking-thunderbird-esr10:
--- → 11+
Reporter | ||
Comment 33•13 years ago
|
||
Works here with todays nightly builds (tried only e-mail component, not the news component). Thanks a lot! Will the fix be also available to the esr branch?
Assignee | ||
Comment 34•13 years ago
|
||
Yes it will be on the ESR for organisations, I forgot to update the bug when I landed it earlier today: http://hg.mozilla.org/releases/comm-esr10/rev/740aa8a7fa8e
status-thunderbird-esr10:
--- → fixed
Comment 35•13 years ago
|
||
For someone who is not a programmer, Status:RESOLVED FIXED means I can use the latest version of Thunderbird with the spell-checker enabled. Can you please send me a link to the build in which this bug is fixed? Thanks.
Assignee | ||
Comment 36•13 years ago
|
||
(In reply to avner.falk@usa.net from comment #35) > For someone who is not a programmer, Status:RESOLVED FIXED means I can use > the latest version of Thunderbird with the spell-checker enabled. Can you > please send me a link to the build in which this bug is fixed? Thanks. The full fix will be in Thunderbird 11. For now you can test this with the beta builds here: http://www.mozilla.org/en-US/thunderbird/channel/ (or you can use the Earlybird ones which are currently version 12, but the fix is there as well).
Comment 37•13 years ago
|
||
Someone maybe should update the Target Milestone - I was going to ask why this fix wasn't going to be available until Thunderbird 13, but am very happy to hear it will be available in 11... Thanks guys!
Assignee | ||
Comment 38•13 years ago
|
||
(In reply to Charles from comment #37) > Someone maybe should update the Target Milestone - I was going to ask why > this fix wasn't going to be available until Thunderbird 13, but am very > happy to hear it will be available in 11... The Target Milestone is the release at which it has landed in trunk. The status-thunderbird<nn> flags are the branch releases it lands in. This bug has both status-thunderbird12 and status-thunderbird11 marked as "fixed", so that gives you the additional information.
Updated•13 years ago
|
Whiteboard: [gs]
You need to log in
before you can comment on or make changes to this bug.
Description
•