Closed Bug 1485317 Opened 6 years ago Closed 6 years ago

Replace 0px with 0 throughout menulist.css

Categories

(Toolkit :: Themes, enhancement, P3)

enhancement

Tracking

()

RESOLVED DUPLICATE of bug 1457216

People

(Reporter: dao, Unassigned, Mentored)

References

Details

(Keywords: good-first-bug)

+++ This bug was initially created as a clone of Bug #1483252 +++

0 doesn't need a unit in CSS since 0px is the same as 0em for instance. So let's remove the "px":

https://searchfox.org/mozilla-central/search?q=+0px&case=true&path=menulist.css
(In reply to Dão Gottwald [::dao] from comment #0)
> +++ This bug was initially created as a clone of Bug #1483252 +++
> 
> 0 doesn't need a unit in CSS since 0px is the same as 0em for instance. So
> let's remove the "px":
> 
> https://searchfox.org/mozilla-central/search?q=+0px&case=true&path=menulist.
> css

I'm new to the open source community and looking to make some first contributions. Can i work on this bug?
Please do. :) Let me know if you have questions.
Assignee: nobody → adithyankv007
I'm new to bugzilla as well,could you please walk me through the workflow of fixing and submitting a bug here?
(In reply to Adithyan K V from comment #3)
> I'm new to bugzilla as well,could you please walk me through the workflow of
> fixing and submitting a bug here?

Have you read a guide like https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch ?
Thank you, I had only worked with git before. Haven't yet worked with mercurial. Will look into it.
Bug 1457216 fixed this. Adithyan K V, let me know if you're looking for another bug to work on.
Assignee: adithyankv007 → nobody
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.