Closed
Bug 1483813
Opened 3 years ago
Closed 3 years ago
Remove .tab-bottom styling
Categories
(Toolkit :: Themes, enhancement, P3)
Toolkit
Themes
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: dao, Assigned: sahilbhosale63, Mentored)
References
Details
(Keywords: good-first-bug, Whiteboard: [lang=css])
Attachments
(1 file)
|
1.47 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
This is unused in mozilla-central, let's remove it. https://searchfox.org/mozilla-central/search?q=tab-bottom&case=true&path=toolkit%2Fthemes
| Assignee | ||
Comment 1•3 years ago
|
||
can I work on this issue??
| Reporter | ||
Comment 2•3 years ago
|
||
(In reply to Sahil Bhosale from comment #1) > can I work on this issue?? Yep.
Assignee: nobody → sahilbhosale63
| Assignee | ||
Comment 3•3 years ago
|
||
Done. Made the requested changes.
Attachment #9001575 -
Flags: review?(dao+bmo)
| Reporter | ||
Comment 4•3 years ago
|
||
Comment on attachment 9001575 [details] [diff] [review] tabbox1.patch Thanks :)
Attachment #9001575 -
Flags: review?(dao+bmo) → review+
| Reporter | ||
Updated•3 years ago
|
Keywords: checkin-needed
| Assignee | ||
Comment 5•3 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #4) > Comment on attachment 9001575 [details] [diff] [review] > tabbox1.patch > > Thanks :) Welcome :)
Pushed by ebalazs@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/92c65ba21514 Remove .tab-bottom styling. r=dao
Keywords: checkin-needed
Comment 7•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/92c65ba21514
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
| Assignee | ||
Comment 8•2 years ago
|
||
Does the patch for this bug has been merged in the latest version of firefox?
Comment 9•2 years ago
|
||
(In reply to Sahil Bhosale from comment #8)
Does the patch for this bug has been merged in the latest version of firefox?
Yes, it was in version 63.
You need to log in
before you can comment on or make changes to this bug.
Description
•