Attachment reminder keywords list opens wrong options tab

RESOLVED FIXED in Thunderbird 17.0


9 years ago
6 years ago


(Reporter: stef, Assigned: aceman)



Thunderbird 17.0
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)



(1 attachment, 1 obsolete attachment)

2.94 KB, patch
: review+
Details | Diff | Splinter Review


9 years ago
When attachment reminder bar opens and one will click on keyword list, the settings window will open on "Composition" but with wrong tab selected if other then the "General" tab was the last open one.
Keywords: polish

1 tools > options > composition: set focus on spelling tab, close options dialogue with ok or cancel
2 write mail with attachment keyword
-> attachment reminder bar pops up (ok)
3 click on linkified keyword to edit list of attachment reminder keywords

- options window opens with focus still on spelling tab
- can't edit keywords list from here
-> confusion for users

- ensure that focus is on composition > general tab, where the button for editing the keyword list is located
OS: Linux → All
Summary: Attachment reminder keyword list opens wrong options tab → Attachment reminder keywords list opens wrong options tab
Bug 718139 Comment 13 experiences the same problem as this bug.
Version: 3.0 → Trunk


7 years ago
Depends on: 718139


7 years ago
Assignee: nobody → acelists

Comment 3

7 years ago
The call seems to be here, it only specifies the pane.

If infrastructure for opening specific pane and tab lands in bug 718139, then this should be made to work easily.
However I wonder why selecting the proper pane (composition) already works... I didn't find the code for that and re-implement it in bug 718139.


7 years ago
Depends on: 360891, 761797

Comment 4

7 years ago
Created attachment 643159 [details] [diff] [review]

I can even open just the attachment keyword editor hiding the full preferences dialog. The patch uses the same approach as bug 360891. Think bwinton had some problem with that on Mac. Magnus Melin, maybe you can look at it too, if you want the full prefs dialog or only the keyword editor (subdialog).
Also this patch must be applied on top of patch from bug 761797 as it changes the same file.
Attachment #643159 - Flags: ui-review?(bwinton)
Attachment #643159 - Flags: review?(mkmelin+mozilla)
Attachment #643159 - Flags: feedback?(mconley)
Comment on attachment 643159 [details] [diff] [review]

So, I'm almost really happy with it…  ;)

The only problem I have is that the attachment keywords show up before the preference dialog does, and it feels really strange.

When you fix that, ui-r=me!  :)

Attachment #643159 - Flags: ui-review?(bwinton) → ui-review+
(Uh, that's on Mac, since it seems like it would make a difference.)

Comment 7

7 years ago
OK, can you see if this patch: from bug 360891 makes the dialogs appear better?
Comment on attachment 643159 [details] [diff] [review]

Review of attachment 643159 [details] [diff] [review]:

Looks good to me, i do like just showing the dialog with the keywords. Sorry for the delay. r=mkmelin

::: mail/components/compose/content/MsgComposeCommands.js
@@ +1901,5 @@
>      msg.setAttribute("flex", "100");
>      msg.onclick = function(event)
>      {
> +      openOptionsDialog("paneCompose", "generalTab", {subdialog_only: "attachment_reminder_button"});

overlong line, wrap after generaltab
Attachment #643159 - Flags: review?(mkmelin+mozilla) → review+

Comment 9

7 years ago
Thanks. Unfortunately bwinton rejected this approach in bug 360891 and wants to see the preferences dialog too. The same in comment 5 here. It probably flickers too much on some setups.

So I need to rework the patch.

Comment 10

6 years ago
Created attachment 652550 [details] [diff] [review]
patch v2

Reworked per bwinton's preference. The advantage of this solution is that the user also sees the "check for missing attachment" option, which wouldn't be the case if only the subdialog was shown.
Attachment #643159 - Attachment is obsolete: true
Attachment #643159 - Flags: feedback?(mconley)
Attachment #652550 - Flags: review?(mkmelin+mozilla)
Comment on attachment 652550 [details] [diff] [review]
patch v2

Review of attachment 652550 [details] [diff] [review]:

Looks good to me! r=mkmelin
Attachment #652550 - Flags: review?(mkmelin+mozilla) → review+


6 years ago
Keywords: checkin-needed
Last Resolved: 6 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
You need to log in before you can comment on or make changes to this bug.