841215 - Change - Ensure selected tab thumbnail is visible when opening tab app bar

Status: VERIFIED FIXED

(Tracking Graveyard :: Metro Operations, defect, P1)

Description

[juan becerra [:juanb]]

Tested on 2013-02-13 Nightly built from http://hg.mozilla.org/project/elm/rev/b3e65fe37681

You can have multiple tabs open and if you happen to glance at the tab strip and you scroll the focused tab thumbnail past the view, and dismiss the strip, going back can be a little confusing because you need to scroll left or right to find it.

Maybe the focused tab should always come up visible in the strip if you happen to go back to the strip after scrolling it out of view.

Comment 1

[Asa Dotzler [:asa]]

Yes, you are right Juan. I thought that's what I specified in the story but I realize the story might not have been clear. Let's treat this like a change. When opening the tab strip, the focused tab should be in the visible area and if that means the strip has to scroll, then it should scroll.

Comment 2

[Frank Yan (:fryn)]

Attachment: patch

I tried to add the handler as a <handler/>, but I guess the event doesn't bubble or something, so I did this instead.

Also, MozContextUIExpand is called both when closing and opening the tab app bar, but this particular handler should only be called when opening the tab strip. It's not a big deal though, and we can revisit this later when the overall UI is more finalized.

Comment 3

[Frank Yan (:fryn)]

Added removeEventListener in a destructor to be safe (following the pattern used in all the other browser/metro/ bindings).

https://hg.mozilla.org/integration/mozilla-inbound/rev/743eb982010a

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/743eb982010a

Comment 5

[juan becerra [:juanb]]

Tested on 2013-02-28 with a nightly build from http://hg.mozilla.org/mozilla-central/rev/c65d59d33aa8
- When I scroll the tab strip beyond the point where the focused tab is, and I dismiss the strip, going back will show me the focused tab highlighted.
- The focused tab will be shown on the left or right, and highlighted, depending on how I scrolled the strip.
- Also tried restarting the browser, having setup "remember tabs from last time", just for kicks, and this still works as expected.

Comment 6

[Samvedana (:Samvedana)]

I can not understand steps to test this. Can any one provide steps to test this?

Comment 7

[Matt Brubeck (:mbrubeck)]

Steps to reproduce/test:
1. Open a large number of tabs, so the tab strip becomes scrollable.
2. Scroll the tab strip so the selected tab is no longer visible.
3. Dismiss the tab strip.
4. Open the tab strip again.

Expected results: The selected tab is visible again after step 4.

Comment 8

[Samvedana (:Samvedana)]

(In reply to Matt Brubeck (:mbrubeck) from comment #7)
> Steps to reproduce/test:
> 1. Open a large number of tabs, so the tab strip becomes scrollable.
> 2. Scroll the tab strip so the selected tab is no longer visible.
> 3. Dismiss the tab strip.
> 4. Open the tab strip again.
> 
> Expected results: The selected tab is visible again after step 4.

Thanks Matt.

User Agent:Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:25.0) Gecko/20130724 Firefox/25.0
Build ID: 20130724030204

WFM.

Tested on Windows 8.1 preview for iteration 10 using latest nightly built from http://hg.mozilla.org/mozilla-central/rev/2983ca6d4d1a

I followed steps provided in comment 7 and got expected result.

Comment 9

[Samvedana (:Samvedana)]

User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130808030205
Built from http://hg.mozilla.org/mozilla-central/rev/fd4cf30428b0

WFM
Tested on windows 8 using latest nightly  for iteration-11. Followed steps provided in comment7 and got expected result.

Comment 10

[Samvedana (:Samvedana)]

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130825030201
Built from http://hg.mozilla.org/mozilla-central/rev/01576441bdc6

WFM
Tested on windows 8 using latest nightly for iteration-12. Followed steps provided in comment7 and got expected result.