Closed
Bug 137836
Opened 22 years ago
Closed 22 years ago
When active tab is closed, "beforeselected" attribute remains on previous tab
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
VERIFIED
FIXED
People
(Reporter: bryner, Assigned: bryner)
References
Details
(Whiteboard: [adt2 rtm] custrtm-)
Attachments
(2 files)
866 bytes,
patch
|
bugs
:
review+
jag+mozilla
:
superreview+
|
Details | Diff | Splinter Review |
901 bytes,
patch
|
mikepinkerton
:
review+
bugzilla
:
superreview+
scc
:
approval+
|
Details | Diff | Splinter Review |
Steps to reproduce: - Open a few tabs in the browser window - Switch to the last tab - Close the last tab Result: The formerly next-to-last tab becomes active, but still has the "beforeselected" attribute set on it. This can interfere with nsITheme-based drawing and possibly match incorrect style rules.
Comment 1•22 years ago
|
||
Oh, that's why my last tab mostly never has the right-side border in Classic :)
Assignee | ||
Comment 2•22 years ago
|
||
This makes sure to clean up the attribute state before the tab is removed.
Comment 3•22 years ago
|
||
Comment on attachment 79572 [details] [diff] [review] patch sr=jag
Attachment #79572 -
Flags: superreview+
Comment 4•22 years ago
|
||
Comment on attachment 79572 [details] [diff] [review] patch r=ben@netscape.com
Attachment #79572 -
Flags: review+
Assignee | ||
Comment 5•22 years ago
|
||
nominating for beta... this causes an ugly visual defect in the classic skin where after you close the last tab, the new last tab will no longer draw the right-side border of the tab.
Keywords: nsbeta1
Comment 6•22 years ago
|
||
Well, not only last tab to be exact, any current tab except the first.
Assignee | ||
Comment 7•22 years ago
|
||
This has been checked into the trunk, keeping bug open for nsbeta1 consideration.
Comment 8•22 years ago
|
||
Comment on attachment 79572 [details] [diff] [review] patch Um... what was wrong with oldTab.selected = false; ? See bug 102110 of which this is a duplicate.
Comment 9•22 years ago
|
||
Nav triage team: nsbeta1+, adt2 rtm
Assignee | ||
Comment 10•22 years ago
|
||
*** Bug 102110 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 11•22 years ago
|
||
Right on both counts. I looked for a dup but this one wasn't apparent from the summary; and what you suggest is indeed cleaner. Follow-up patch coming up.
Assignee | ||
Comment 12•22 years ago
|
||
Comment 13•22 years ago
|
||
Comment on attachment 80872 [details] [diff] [review] cleanup per neil's suggestion r=pink
Attachment #80872 -
Flags: review+
Comment 14•22 years ago
|
||
Comment on attachment 80872 [details] [diff] [review] cleanup per neil's suggestion sr=blake
Attachment #80872 -
Flags: superreview+
Comment 15•22 years ago
|
||
Bug 102110 probably wasn't specifc enough to match your query.
Assignee | ||
Comment 16•22 years ago
|
||
Checked into the trunk. Thanks Neil!
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 17•22 years ago
|
||
Isn't this fix going into the branch? You know it causes (or is) bug 116394, which is about the missing borders of tabs after closing others...
Comment 18•22 years ago
|
||
Nominating for mozilla1.0 (simple fix, fixes an ugly visual defect)
Keywords: mozilla1.0
Comment 19•22 years ago
|
||
adt1.0.0+ (on ADT's behalf) for approval to checkin to the 1.0 branch, pending Drivers' approval. After, checking in, please add the fixed1.0 keyword.
Comment 20•22 years ago
|
||
Comment on attachment 80872 [details] [diff] [review] cleanup per neil's suggestion a=scc (on behalf of drivers) for checkin to the mozilla1.0 branch; please get it in asap
Attachment #80872 -
Flags: approval+
Updated•22 years ago
|
Whiteboard: [adt2 rtm] → [adt2 rtm] custrtm-
Comment 22•21 years ago
|
||
verif fixd 0812 brnch/trunk.
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.0 → verified1.0.0
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: shrir → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•