Drop down list in Customize Toolbar is too small



4 years ago
4 years ago


(Reporter: sztsian, Assigned: bv1578)


Lightning 3.3
Dependency tree / graph



(1 attachment, 1 obsolete attachment)

2.27 KB, patch
: review+
Details | Diff | Splinter Review
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.
This also exists on Solaris.
Component: Untriaged → Lightning Only
Product: Thunderbird → Calendar
Version: 31 → Lightning 3.3
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.
Ever confirmed: true
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: nobody → bv1578
Posted patch patch - v1 (obsolete) — Splinter Review
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-
Posted patch patch - v1Splinter Review
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+
Keywords: checkin-needed
Pushed to https://hg.mozilla.org/comm-central/rev/3fe65676c36b
Closed: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.3
Depends on: 1208704
You need to log in before you can comment on or make changes to this bug.