113208 - CTRL-TAB in nested tabbox switches parent tab

Status: RESOLVED INVALID

(Core :: XBL, defect)

Description

[tan]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.6+) Gecko/20011129
BuildID:    2001-11-29

in a nested tabbox layout, pressing ctrl w switches ALL tab panels up to the parent.

Reproducible: Always
Steps to Reproduce:
1.Make an XUL page with nested tabboxes 
2.Select a nested tab panel
3.press CTRL-W

Actual Results:  All panels are switched

Expected Results:  tab panel farthest down the heirarchy of selected (and
focused) tab panels should switch. i.e. event bubbling should be cancelled.

(I don't know which component this should belong to, but I would make a guess
that its a XBL thing... possibly)

Comment 1

[John Morrison]

Attachment: testcase; nested tabbox and panels

Comment 2

[John Morrison]

worksforme

Comment 3

[tan]

ahhh... i see. you put your nested tabboxes inside a <vbox> tag.

I was doing this: 
<tabpanels>
  <tabbox>
   ...
  </tabbox>
  <tabbox>
   ...
  </tabbox>
</tabpanels>

Although, I'm certain its entirely legal XUL as it does the same thing. My
feeling is the <vbox> intercepts the CTRL-W key event and swallows it, but
without the surrounding <vbox> tag, the CTRL-W key event propagates up to its
parent. 

Ah well. Thanks! I'll use <box> around everything from now on.

ps. Interestingly, CTRL-W also closes the navigator window -- is this a conflict
of interest? 

ie. 
1. start up mozilla
2. load file:///home/tabbox.xul in mozilla
3. press ctrl-w 

mozilla closes, instead of switching tab panes. i guess that is intended?

thanks again!
tan.

Comment 4

[John Morrison]

I don't know why but I didn't really read this (even though it says Ctrl-W
in the summary). 

Where is Ctrl-W ever supposed to switch tabs? (It doesn't matter that they are 
in a vbox or tabbox as the firstChild by the way). Ctrl-W closes windows, and
as far as I can see there is no key binding that is attempting to override this 
for tabbox.

Comment 5

[John Morrison]

this isn't valid

Comment 6

[tan]

sorry i meant CTRL-TAB