Closed Bug 637126 Opened 14 years ago Closed 9 years ago

misspositioned tabbar arrow with only apptabs open

Categories

(Firefox Graveyard :: Panorama, defect, P3)

x86
Windows 7
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: robhau, Unassigned)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows NT 6.1; rv:2.0b12) Gecko/20100101 Firefox/4.0b12
Build Identifier: Mozilla/5.0 (Windows NT 6.1; rv:2.0b12) Gecko/20100101 Firefox/4.0b12

When you drag a tab from another firefox window into one with only apptabs opened, the little arrow indicating the future tab position will appear mostleft in the window, before the apptabs

Reproducible: Always

Steps to Reproduce:
1.Transform all open tabs except of one into apptabs
2.drag the normal tab out of firefox to open him into a new window
3.now drag him back into the apptab-window, right to the apptabs.

Actual Results:  
the little arrow will appear left from the firefox-menubutton

Expected Results:  
arrow should be in the right of apptabs
Summary: mispositioned tabbar arrow whit only apptabs open → misspositioned tabbar arrow with only apptabs open
Can you reproduce that in safe mode with all add-ons disabled?
Version: unspecified → Trunk
yes
Works For Me: 
Mozilla/5.0 (Windows NT 6.1; rv:2.0b12) Gecko/20100101 Firefox/4.0b12

*Note: Can you attach some screenshots with the issue? Following the steps to reproduce did not result in any issue. The arrow appears where the user drags the tab to position it in the window with apptabs only
There you are 
http://robhau.de/stuff/scrnsht.jpg
It looks like an important issue.
Can you reproduce it on a clean profile (with no history, bookmarks et cetera) or in safe mode?
http://support.mozilla.com/kb/Safe+Mode
http://support.mozilla.com/kb/Managing+profiles
I tested it, the issue only appears when using TabCandy in main Window, doesnt matter whether the profile is clean or not.
Anyway its not such a bad bug, as long a the tab gets inserted correctly.
But  after all it should be fixed.
Component: Tabbed Browser → TabCandy
Reproduceable on:
Mozilla/5.0 (Windows NT 6.1; rv:2.0b12) Gecko/20100101 Firefox/4.0b12

And on:
Mozilla/5.0 (Windows NT 6.1; rv:2.0b13pre) Gecko/20110301 Firefox/4.0b13pre

*Notes: 
Here are more clear STRs, if needed:
1. With FFx open have at least 2 Groups (in Panorama) and 4 Apptabs available.
2. Focus on one of the groups 
3. Leave one single tab beside the Apptab, in that group
4. Drag that tab out to get another window
5. Try to drag and drop the tab back to the previous window (that was initially in), to the right of the apptab.

Expected result: the pointer arrow should be properly displayed.

Actual result: please refer to Comment 4 (see attachement)
QA Contact: tabbed.browser → tabcandy
I can repro. That's really odd.
Blocks: 585689
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Attached patch v1 (deleted) — Splinter Review
Attachment #516544 - Flags: review?(dao)
Attachment #516544 - Flags: feedback?(ian)
Comment on attachment 516544 [details] [diff] [review]
v1

I'm sorry, I don't know enough about this code to give meaningful feedback.
Attachment #516544 - Flags: feedback?(ian)
No longer blocks: 585689
Assignee: nobody → raymond
Status: NEW → ASSIGNED
bugspam
No longer blocks: 603789
Tested with latest trunk and this patch still works.  

Dao: please give some feedback on this when you have time.
bugspam
No longer blocks: 653099
bugspam
Blocks: 660175
bugspam

(Fx7 was branched, removing open bugs from Fx7 meta bug, we won't create new meta bugs for upcoming Fx versions)
No longer blocks: 660175
Status: ASSIGNED → NEW
Assignee: raymond → nobody
Comment on attachment 516544 [details] [diff] [review]
v1

Not sure what this patch does, other than using visibleTabs instead of childNodes. Seems like a larger refactoring that unfortunately makes the code harder to follow, which is why this was sitting in my review queue for ages (sorry). Needs some documentation.
Attachment #516544 - Flags: review?(dao)
Panorama has been removed from Firefox 45, currently in Beta and scheduled for release on March 7th. As such, I'm closing all existing Panorama bugs.

If you are still using Panorama, you will see a deprecation message in Firefox 44, and when 45 is released your tab group data will be migrated to bookmarks, with a folder for each group. There are also a few addons offering similar functionality.

See https://support.mozilla.org/en-US/kb/tab-groups-removal for more info.

We're removing Panorama because it has extremely low usage (about 0.01% of users), and has a large number of bugs and usability issues. The cost of fixing all those issues is far too high to justify, and so we'll instead be focusing our time and energy on improving other parts of Firefox.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: