Closed Bug 1305764 Opened 4 years ago Closed 4 years ago

Missing rating-unrated.png referenced from chrome://mozapps/skin/extensions/extensions.css

Categories

(Toolkit :: Add-ons Manager, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: florian, Unassigned, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [lang=css] triaged)

Attachments

(2 files, 1 obsolete file)

From bug 1221383:
missing chrome://mozapps/skin/extensions/rating-unrated.png referenced from chrome://mozapps/skin/extensions/extensions.css

Bug 601022 removed the rating-unrated.png file on all platforms, but kept the reference in the Linux extensions.css file:

file removal for linux: https://hg.mozilla.org/mozilla-central/rev/367398a8cbfe#l22.27

removal of the rule on Mac: https://hg.mozilla.org/mozilla-central/rev/367398a8cbfe#l30.928

removal of the rule on Windows: https://hg.mozilla.org/mozilla-central/rev/367398a8cbfe#l51.954

All we need to do here is remove these lines:
.meta-rating[showrating="user"] > .star {
  list-style-image: url("chrome://mozapps/skin/extensions/rating-unrated.png");
  padding: 2px 3px;
}

at http://searchfox.org/mozilla-central/rev/767e1e9b118269f957ca1817138472146278a29e/toolkit/themes/linux/mozapps/extensions/extensions.css#38
Priority: -- → P5
Whiteboard: [lang=css] → [lang=css] triaged
Hi,

I am Abhinav Srivastava and I've been working on HTML/CSS/JavaScript from last one and half years. So, I would like to work on this bug, please guide me.


Thanks & Regards,
Abhinav
Hi Abhinav,

Have you already downloaded the Firefox source code? Once you have, you can follow the instructions at the end of the bug description here (click the last link to see where the lines that need to be removed are). Let me know if you have questions :-).
Hi,

I am Rajat Garg I've downloaded the Firefox source code and have build it on my machine.I have made the changes required for this bug locally.What next I have to do merge the change.

Thanks & Regards,
Rajat
Comment on attachment 8797641 [details] [diff] [review]
Removed the specified lines from specified file

Review of attachment 8797641 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the patch! The code change looks good, but the patch isn't in a format suitable for being checked in. It looks like you created this diff using the diff command rather than using hg export or hgbzexport.

See https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Your patch file should contain a commit message and the name/email of the author.
Attachment #8797641 - Flags: review?(florian) → feedback+
Hi,
   I mistakenly pushed the wrong change in my first commit and then corrected that in second and removed the  reference of the missing file too but now how do i squash them or delete the wrong changes.What should i do next?

Thanks
Rajat garg
The hg histedit command supports folding changesets. See https://www.mercurial-scm.org/wiki/HisteditExtension

Note: in commit messages we usually only take the IRC nick of the reviewer, so for me it would be r=florian
Attachment #8797678 - Attachment is obsolete: true
Attachment #8797678 - Flags: review?(florian)
Comment on attachment 8797677 [details]
Bug 1305764 Removed the rating-unrated.png refrence from linux extensions.css file.

https://reviewboard.mozilla.org/r/83322/#review82186

Looks good, thanks!

Note: when folding the 2 changesets, mercurial concatenated the commit messages.
Attachment #8797677 - Flags: review?(florian) → review+
If you would be interested in working on another bug similar to this one, there's bug 1305748 that you can have a look at.
(In reply to Florian Quèze [:florian] [:flo] from comment #13)
> If you would be interested in working on another bug similar to this one,
> there's bug 1305748 that you can have a look at.

Thanks florian for helping me out in solving my first bug.And thanks to IRC channel too.I would love to work on another bug.
Rajat garg
https://hg.mozilla.org/integration/fx-team/rev/f732bc68879c82010aae7d030483dd5985c69f75
Bug 1305764 - Removed the rating-unrated.png refrence from linux extensions.css file. r=florian
https://hg.mozilla.org/mozilla-central/rev/f732bc68879c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.