Closed Bug 1081203 Opened 11 years ago Closed 11 years ago

Remove dead #social-toolbar-item CSS since there is nothing called social-toolbar-item defined outside of the CSS

Categories

(Firefox :: Theme, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 36

People

(Reporter: jaws, Assigned: himanshusingh.singh06, Mentored)

Details

(Whiteboard: [good first bug][lang=css])

Attachments

(1 file, 2 obsolete files)

http://mxr.mozilla.org/mozilla-central/search?string=social-toolbar-item should show usages outside of CSS if it is going to be used within CSS. Currently, it shows: > social-toolbar-item > > Found 10 matching lines > /browser/themes/osx/browser.css (View Hg log or Hg annotations) > > line 4371 -- #social-toolbar-item > .toolbarbutton-1 { > line 4380 -- #social-toolbar-item > .toolbarbutton-1:-moz-locale-dir(ltr) { > line 4383 -- #social-toolbar-item > .toolbarbutton-1:last-child:-moz-locale-dir(ltr) { > line 4386 -- #social-toolbar-item > .toolbarbutton-1:-moz-locale-dir(rtl) { > line 4389 -- #social-toolbar-item > .toolbarbutton-1:first-child:-moz-locale-dir(rtl) { > line 4393 -- #social-toolbar-item > .toolbarbutton-1:-moz-locale-dir(ltr):first-child, > line 4394 -- #social-toolbar-item > .toolbarbutton-1:-moz-locale-dir(rtl):last-child { > line 4399 -- #social-toolbar-item > .toolbarbutton-1:-moz-locale-dir(rtl):first-child, > line 4400 -- #social-toolbar-item > .toolbarbutton-1:-moz-locale-dir(ltr):last-child { > line 4406 -- #social-toolbar-item > toolbaritem { > > Found 10 matching lines
Assuming you're right about this being a useless rule, this will make a nice good-first-bug.
Mentor: jaws
Whiteboard: [good first bug][lang=css]
Hey id like to work on this bug. How do i begin?
Flags: needinfo?(jaws)
If it is possible I would like to work on this bug as it would be my first? :)
So is for https://mxr.mozilla.org/mozilla-central/search?string=social-provider-button, this also doesn't seem to be used anywhere outside of CSS.
Hi I would like to work on this bug how do I start?
Hi I already have the builds set-up . Please could I work on this bug? Thanks Jesal
Attached patch 1081203.patch (obsolete) — Splinter Review
Attachment #8503776 - Flags: review?(jaws)
I would like to be assigned to this bug, Thank you
Comment on attachment 8503776 [details] [diff] [review] 1081203.patch Review of attachment 8503776 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, this looks good. Can you also remove the following rules (as pointed out in comment #4): https://mxr.mozilla.org/mozilla-central/search?string=social-provider-button
Attachment #8503776 - Flags: review?(jaws) → feedback+
Assignee: nobody → himanshusingh.singh06
Status: NEW → ASSIGNED
Flags: needinfo?(jaws)
Attached patch 1081203.patch (obsolete) — Splinter Review
Attachment #8503776 - Attachment is obsolete: true
Attachment #8504465 - Flags: review?(jaws)
Comment on attachment 8504465 [details] [diff] [review] 1081203.patch Review of attachment 8504465 [details] [diff] [review]: ----------------------------------------------------------------- This is close but it is missing the removal of the #social-provider-button rules from the /browser/themes/osx/browser.css stylsheet.
Attachment #8504465 - Flags: review?(jaws) → feedback+
Attached patch 1081203.patchSplinter Review
Attachment #8504465 - Attachment is obsolete: true
Attachment #8504668 - Flags: review?(jaws)
Comment on attachment 8504668 [details] [diff] [review] 1081203.patch Review of attachment 8504668 [details] [diff] [review]: ----------------------------------------------------------------- Looks good! Thanks! I'll mark this as checkin-needed now, and it should be checked in to the fx-team integration branch within a day or two. Once it gets there, then it will get merged to mozilla-central within the next day or two. (All of this usually happens faster, but sometimes there are delays if patches introduce test failures). Once it's in the mozilla-central repository, it will show up in Firefox Nightly the next day. Congratulations on your first patch. Thanks for sticking with it.
Attachment #8504668 - Flags: review?(jaws) → review+
Keywords: checkin-needed
Whiteboard: [good first bug][lang=css] → [good first bug][lang=css][fixed-in-fx-team]
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #13) > Congratulations on your first patch. Thanks for sticking with it. That landed now in fx-team! Thanks for contributing to Mozilla!
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][lang=css][fixed-in-fx-team] → [good first bug][lang=css]
Target Milestone: --- → Firefox 36
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: