1162380 - Drop down list in Customize Toolbar is too small

Status: RESOLVED FIXED

(Calendar :: Lightning Only, defect)

Description

[Zamir SUN]

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

[sun.ziqian]

This also exists on Solaris.

Comment 2

[[:MakeMyDay]]

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.

Comment 3

[[:MakeMyDay]]

Philipp, do you know what attribute triggers pre-setting of the dropdown selection?

Comment 4

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

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.

Comment 5

[Decathlon]

Attachment: 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?

Comment 6

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

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.

Comment 7

[Decathlon]

Attachment: patch - v1

Sometimes it helps doing a qrefresh (and verify the file before posting) ;-)

Comment 8

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

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

Comment 9

[Stefan Sitter [:ssitter]]

Pushed to https://hg.mozilla.org/comm-central/rev/3fe65676c36b