Closed Bug 581286 Opened 14 years ago Closed 14 years ago

Don't duplicate tabbox.advanceSelectedTab and key handling code

Categories

(Firefox Graveyard :: Panorama, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

Attachments

(1 file)

Bug 565968 has a some code from tabbox to handle switching tabs. We shouldn't duplicate the code and just reuse it.
Depends on: 579222
http://hg.mozilla.org/users/edward.lee_engineering.uiuc.edu/tabcandy-central/rev/15284adb0a68
Prevent tabs from showing up in the tabbar by hiding them instead of collapsing so that existing tabbox code knows to skip over them. Remove key handling code except for activating tabcandy and switching groups.
Assignee: nobody → edilee
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Attached patch v1Splinter Review
Switched from collapsed to hidden.
Attachment #459734 - Flags: feedback?(ian)
Comment on attachment 459734 [details] [diff] [review]
v1

Looks good.
Attachment #459734 - Flags: feedback?(ian) → feedback+
Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy.  Filter the bugmail spam with "tabcandymassmove".
Product: Mozilla Labs → Firefox
Target Milestone: -- → ---
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: