List all tabs dropdown is empty with Lightning installed

RESOLVED FIXED

Status

Calendar
Lightning: SeaMonkey Integration
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: InvisibleSmiley, Assigned: Karsten Düsterloh)

Tracking

({fixed-seamonkey2.0})

Trunk
fixed-seamonkey2.0

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
[Current SM2 nightly, multiple MailNews tabs open]

When Lightning is installed the List all tabs dropdown at the right end of the MailNews tab bar is empty. Without Lightning the dropdown shows one row per open tab.

Without Lightning installed the JavaScript Debugger (using Stop and F12 while hovering/clicking the List all tabs button) eventually enters tabmail.xml line 1441 which is the tabmail-alltabs-popup binding's popupshowing handler (Call Stack shows only this line, name = onxblpopupshowing). With Lightning that line is not reached, instead it goes directly to the popup binding's popupshowing handler (global popup.xml line 181) which, if without Lightning, is called after the binding in tabmail.xml. Something seems to be messing with popupshowing calling/registration...
(Assignee)

Comment 1

8 years ago
Created attachment 406107 [details] [diff] [review]
override toolkit's .menulist-menupopup binding
Assignee: nobody → mnyromyr
Attachment #406107 - Flags: ui-review?(neil)
Attachment #406107 - Flags: review?(neil)

Comment 2

8 years ago
Comment on attachment 406107 [details] [diff] [review]
override toolkit's .menulist-menupopup binding

r+sr=me (it doesn't feel right to set the ui-r flag for sr!)
Attachment #406107 - Flags: ui-review?(neil)
Attachment #406107 - Flags: review?(neil)
Attachment #406107 - Flags: review+

Comment 3

8 years ago
Comment on attachment 406107 [details] [diff] [review]
override toolkit's .menulist-menupopup binding

approval-seamonkey2.0=me

Comment 4

8 years ago
I don't see the problem with either:

Build identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; rv:1.9.1.5pre) Gecko/20091013 Lightning/1.0pre SeaMonkey/2.0pre

or

Build identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; rv:1.9.1.5pre) Gecko/20091008 Lightning/1.0pre SeaMonkey/2.0pre

Comment 5

8 years ago
I see it definitely with Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.5pre) Gecko/20091013 Lightning/1.0pre SeaMonkey/2.0pre ID:20091013002653

the Lightning build is also from 20091013

The only difference I see is I have Vista you have WinXP.
(Reporter)

Comment 6

8 years ago
(In reply to comment #1)
> Created an attachment (id=406107) [details]
> override toolkit's .menulist-menupopup binding

That does it. :-)

[sr: comment 2, a: comment 3]
Status: NEW → ASSIGNED
Keywords: checkin-needed
(Assignee)

Comment 7

8 years ago
Pushed as <http://hg.mozilla.org/comm-central/rev/b6b033026b70>.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed → fixed-seamonkey2.0
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.