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

RESOLVED FIXED in Firefox 52

Status

()

Toolkit
Add-ons Manager
P5
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: florian, Unassigned, Mentored)

Tracking

({good-first-bug})

Trunk
mozilla52
good-first-bug
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

(Whiteboard: [lang=css] triaged)

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

a year ago
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

Comment 1

a year ago
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
(Reporter)

Comment 2

a year ago
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 :-).

Comment 3

a year ago
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 5

a year ago
Created attachment 8797641 [details] [diff] [review]
Removed the specified lines from specified file
Attachment #8797641 - Flags: review?(florian)
(Reporter)

Comment 6

a year ago
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+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 9

a year ago
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
(Reporter)

Comment 10

a year ago
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
Comment hidden (mozreview-request)

Updated

a year ago
Attachment #8797678 - Attachment is obsolete: true
Attachment #8797678 - Flags: review?(florian)
(Reporter)

Comment 12

a year ago
mozreview-review
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+
(Reporter)

Comment 13

a year ago
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.

Comment 14

a year ago
(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
(Reporter)

Comment 15

a year ago
https://hg.mozilla.org/integration/fx-team/rev/f732bc68879c82010aae7d030483dd5985c69f75
Bug 1305764 - Removed the rating-unrated.png refrence from linux extensions.css file. r=florian

Comment 16

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f732bc68879c
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox52: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.