Drop down list in Customize Toolbar is too small

RESOLVED FIXED in 4.3

Status

Calendar
Lightning Only
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Zamir SUN, Assigned: Decathlon)

Tracking

Lightning 3.3

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.27 KB, patch
Fallen
: review+
Details | Diff | Splinter Review
(Reporter)

Description

2 years ago
User Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:36.0) Gecko/20100101 Firefox/36.0
Build ID: 20150220131016

Steps to reproduce:

1. Install Thunderbird 31.6 and lightning 3.3.3
2. Go to Calendar view
3. Right click on menu bar, choose Customize
4. See the drop down list in "Show"



Actual results:

The drop down list is blank by default and is not the same height with "Show"


Expected results:

The drop down list should be the same height with "Show" as when we trigger it on main view.

Comment 1

2 years ago
This also exists on Solaris.

Updated

2 years ago
Component: Untriaged → Lightning Only
Product: Thunderbird → Calendar
Version: 31 → Lightning 3.3

Comment 2

2 years ago
The same appears in Daily. If you selcet an item from the dropdown list and closes the dialog and than reopen the customize dialog, the previous selection is only persisted as you haven't switched to another tab. The same applies on task view.

I'm wondering if there's any attribute/property we're missing in the toolbar definitions in lightning-toolbar.xul which triggers that behaviour or is this a toolkit issue.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 3

2 years ago
Philipp, do you know what attribute triggers pre-setting of the dropdown selection?
Flags: needinfo?(philipp)
Ah I see. So we have this modebox binding that sets the "mode" attribute to whatever mode is active (calendar,mail,tasks). The toolbox uses the mode attribute to determine what mode it is in, for example the mail toolbar has the attribute set to "full" for me.

I don't know why the mode attribute is being modified on that node, I thought it only happens on modeboxes.
Flags: needinfo?(philipp)
(Assignee)

Updated

2 years ago
Assignee: nobody → bv1578
Status: NEW → ASSIGNED
(Assignee)

Comment 5

2 years ago
Created attachment 8608857 [details] [diff] [review]
patch - v1

In the toolbox the attribute "mode", related to the Lightning's current mode (mail, calendar, task), overwrites the attribute "mode" used to set the value in the drop down menu (full, icons, text, textbesideicon). This was my error from another patch.

In this patch I've used modeboxes without set the attribute "refcontrol" because the attribute "collapsed" in the toolbars already allows to hide the toolbar for each mode. Could it be a problem for the modeboxes a missing refcontrol?
I've also changed the order of two items in the context menu for the menu bar so we have the same context menu of the toolbar.

I wanted to test on SeaMonkey, but I'm not able to find a recent version for Windows (zip version, without installer) with a working Lightning. What version should I use, and where I can download it?
Attachment #8608857 - Flags: review?(philipp)
Comment on attachment 8608857 [details] [diff] [review]
patch - v1

Looks like the patch is empty? I just see the header.

I believe Seamonkey is suffering issues with their nightly builds at the moment, their tree has been closed for a while.
Attachment #8608857 - Flags: review?(philipp) → review-
(Assignee)

Comment 7

2 years ago
Created attachment 8608935 [details] [diff] [review]
patch - v1

Sometimes it helps doing a qrefresh (and verify the file before posting) ;-)
Attachment #8608857 - Attachment is obsolete: true
Attachment #8608935 - Flags: review?(philipp)
Comment on attachment 8608935 [details] [diff] [review]
patch - v1

Review of attachment 8608935 [details] [diff] [review]:
-----------------------------------------------------------------

As long as it works in Thunderbird it should be fine, we can wait for a bug report in Seamonkey. r=philipp
Attachment #8608935 - Flags: review?(philipp) → review+
(Assignee)

Updated

2 years ago
Keywords: checkin-needed

Comment 9

2 years ago
Pushed to https://hg.mozilla.org/comm-central/rev/3fe65676c36b
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.3

Updated

2 years ago
Depends on: 1208704
You need to log in before you can comment on or make changes to this bug.