1485317 - Replace 0px with 0 throughout menulist.css

Status: RESOLVED DUPLICATE of bug 1457216

(Toolkit :: Themes, enhancement, P3)

Description

[Dão Gottwald [:dao]]

+++ 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

Comment 1

[Adithyan K V]

(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?

Comment 2

[Dão Gottwald [:dao]]

Please do. :) Let me know if you have questions.

Comment 3

[Adithyan K V]

I'm new to bugzilla as well,could you please walk me through the workflow of fixing and submitting a bug here?

Comment 4

[Dão Gottwald [:dao]]

(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 ?

Comment 5

[Adithyan K V]

Thank you, I had only worked with git before. Haven't yet worked with mercurial. Will look into it.

Comment 6

[Dão Gottwald [:dao]]

Bug 1457216 fixed this. Adithyan K V, let me know if you're looking for another bug to work on.