1483252 - Replace 0px with 0 throughout menu.css

Status: RESOLVED FIXED

(Toolkit :: Themes, enhancement, P3)

Description

[Dão Gottwald [::dao]]

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=menu.css

Comment 1

[Bojan]

Hi, I would like to work on this one. I do not yet have the Firefox source code, so I would probably need to start off with that.

Comment 2

[Dão Gottwald [::dao]]

(In reply to Bojan from comment #1)
> Hi, I would like to work on this one. I do not yet have the Firefox source
> code, so I would probably need to start off with that.

You can find instructions here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Source_Code/Mercurial

Comment 3

[Bojan]

OK, I followed the guide and installed Mercurial, then I pulled the Firefox source code from https://hg.mozilla.org/mozilla-central/, however, since I am coming from Git, I am now not sure how to proceed next. This would be my first contribution to Firefox's source code :)

Comment 4

[Tim Nguyen :ntim]

(In reply to Bojan from comment #3)
> OK, I followed the guide and installed Mercurial, then I pulled the Firefox
> source code from https://hg.mozilla.org/mozilla-central/, however, since I
> am coming from Git, I am now not sure how to proceed next. This would be my
> first contribution to Firefox's source code :)

This might be helpful: https://gist.github.com/cortesben/016cd401faae5a8dae59

Comment 5

[Bojan]

Thanks Tim and Dao for the great help! I was able to make changes, and configured Mercurial through mach, however, when I commited and pushed the changes to a review repo using ```hg push review```, I ended up getting the message ```MozReview is no longer accepting new review requests```

However, my changes did end up here: https://reviewboard-hg.mozilla.org/gecko/rev/4cf83a05b94b

Does that look good, or did I miss something?

Comment 6

[Dão Gottwald [::dao]]

(In reply to Bojan from comment #5)
> Thanks Tim and Dao for the great help! I was able to make changes, and
> configured Mercurial through mach, however, when I commited and pushed the
> changes to a review repo using ```hg push review```, I ended up getting the
> message ```MozReview is no longer accepting new review requests```
> 
> However, my changes did end up here:
> https://reviewboard-hg.mozilla.org/gecko/rev/4cf83a05b94b
> 
> Does that look good, or did I miss something?

Sorry for the confusion. We're in the process of transitioning away from ReviewBoard / MozReview.

Your patch looks good except that your mozilla-central copy is now slightly out of date because bug 1482609 landed in the meantime:

applying https://reviewboard-hg.mozilla.org/gecko/raw-rev/4cf83a05b94b
patching file toolkit/themes/linux/global/menu.css
Hunk #2 FAILED at 90

Comment 7

[Bharat Raghunathan]

Can you please review the patch I made at https://phabricator.services.mozilla.com/D3689
And will patches show up as an attachment here after migration out of MozReview is finished?

Comment 8

[Bharat Raghunathan]

Attachment: Bug 1483252 - Replace 0px with 0 throughout menu.css

MozReview-Commit-ID: DiHVfBuSH7V

Comment 9

[Dão Gottwald [::dao]]

Comment on attachment 9002212 [details]
Bug 1483252 - Replace 0px with 0 throughout menu.css

Dão Gottwald [::dao] has approved the revision.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/060ff053ae27
Replace 0px with 0 throughout menu.css r=dao

Comment 11

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/060ff053ae27