The default bug view has changed. See this FAQ.

Attachment reminder keywords list opens wrong options tab

RESOLVED FIXED in Thunderbird 17.0

Status

Thunderbird
Message Compose Window
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: stef, Assigned: aceman)

Tracking

({polish})

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.94 KB, patch
Magnus Melin
: review+
Details | Diff | Splinter Review
(Reporter)

Description

8 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
STR

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

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

expected
- 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
(Assignee)

Updated

5 years ago
Depends on: 718139
(Assignee)

Updated

5 years ago
Assignee: nobody → acelists
(Assignee)

Comment 3

5 years ago
The call seems to be here, it only specifies the pane.
http://mxr.mozilla.org/comm-central/source/mail/components/compose/content/MsgComposeCommands.js#1934

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.
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
Depends on: 360891, 761797
(Assignee)

Comment 4

5 years ago
Created attachment 643159 [details] [diff] [review]
patch

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]
patch

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!  :)

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

Comment 7

5 years ago
OK, can you see if this patch: https://bugzilla.mozilla.org/attachment.cgi?id=644488 from bug 360891 makes the dialogs appear better?

Comment 8

5 years ago
Comment on attachment 643159 [details] [diff] [review]
patch

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+
(Assignee)

Comment 9

5 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.
(Assignee)

Comment 10

5 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 11

5 years ago
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+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/5322ef0f9d8e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 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.