These are unnecessary since our buttons and menulists don't have a default border anymore: https://searchfox.org/mozilla-central/rev/88439dc6c5b02e167032374071cdc697a056efa1/browser/themes/shared/translation/infobar.inc.css#36-40

Hello, I would like to work on this bug, I am new to organization please help me get started.

(In reply to Ashish Verma [:ashish1500616] from comment #1) > Hello, I would like to work on this bug, I am new to organization please > help me get started. Have you built Firefox yet? https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Simple_Firefox_build Have you read the documentation on how to submit patches? https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch

(In reply to Dão Gottwald [::dao] from comment #2) > (In reply to Ashish Verma [:ashish1500616] from comment #1) > > Hello, I would like to work on this bug, I am new to organization please > > help me get started. > > Have you built Firefox yet? > https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/ > Build_Instructions/Simple_Firefox_build > > Have you read the documentation on how to submit patches? > https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/ > How_to_Submit_a_Patch No, I am building it ri8 now , which version should I choose Firefox for Desktop Artifact Mode or Firefox for Desktop

(In reply to Ashish Verma [:ashish1500616] from comment #3) > No, I am building it ri8 now , which version should I choose Firefox for > Desktop Artifact Mode or Firefox for Desktop Artifact mode is sufficient for front-end bugs like this one, and it builds much faster!

(In reply to Dão Gottwald [::dao] from comment #4) > (In reply to Ashish Verma [:ashish1500616] from comment #3) > > No, I am building it ri8 now, which version should I choose Firefox for > > Desktop Artifact Mode or Firefox for Desktop > > Artifact mode is sufficient for front-end bugs like this one, and it builds > much faster! Hello, I have successfully built firefox on my machine what should be the next step.

Next you should open browser/themes/shared/translation/infobar.inc.css in the editor of your choice and find the lines I linked to in comment 0.

(In reply to Dão Gottwald [::dao] from comment #6) > Next you should open browser/themes/shared/translation/infobar.inc.css in > the editor of your choice and find the lines I linked to in comment 0. Should I comment or completely delete these lines ie from line no 36 to line no 40.

(In reply to Ashish Verma [:ashish1500616] from comment #7) > (In reply to Dão Gottwald [::dao] from comment #6) > > Next you should open browser/themes/shared/translation/infobar.inc.css in > > the editor of your choice and find the lines I linked to in comment 0. > > Should I comment or completely delete these lines ie from line no 36 to line > no 40. Please delete them.

(In reply to Dão Gottwald [::dao] from comment #8) > (In reply to Ashish Verma [:ashish1500616] from comment #7) > > (In reply to Dão Gottwald [::dao] from comment #6) > > > Next you should open browser/themes/shared/translation/infobar.inc.css in > > > the editor of your choice and find the lines I linked to in comment 0. > > > > Should I comment or completely delete these lines ie from line no 36 to line > > no 40. > > Please delete them. And then I will have to submit a patch for review.?

Yes :)

(In reply to Dão Gottwald [::dao] from comment #10) > Yes :) Hello Dao, When I am trying to push I am getting this error pushing to https://reviewboard-hg.mozilla.org/autoreview searching for appropriate review repository redirecting push to https://reviewboard-hg.mozilla.org/gecko abort: you must set mozilla.ircnick in your hgrc config file to your IRC nickname in order to perform code reviews. I have added this to hgrc file. [mozilla] ircnick = mynick

Where is your hgrc file saved?

(In reply to Ashish Verma [:ashish1500616] from comment #11) > (In reply to Dão Gottwald [::dao] from comment #10) > > Yes :) > > Hello Dao, > When I am trying to push > I am getting this error > > pushing to https://reviewboard-hg.mozilla.org/autoreview > searching for appropriate review repository > redirecting push to https://reviewboard-hg.mozilla.org/gecko > abort: you must set mozilla.ircnick in your hgrc config file to your IRC > nickname in order to perform code reviews. > > I have added this to hgrc file. > [mozilla] > ircnick = mynick Now there is a different error root@ashishverma:/media/ashishverma/mozilla-central# hg push review pushing to https://reviewboard-hg.mozilla.org/autoreview searching for appropriate review repository redirecting push to https://reviewboard-hg.mozilla.org/gecko (ignoring public changeset c4e4613dbe32 in review request) abort: no non-public changesets left to review (add or change the -r argument to include draft changesets)

Did you commit something before pushing?

No I dont think so,How to reset this ?

You need to run hg commit like this: hg commit -m "Bug 1429716 - Stop resetting border properties for the translation info bar's buttons and menulists. r=dao"

Comment on attachment 8942233 [details] Bug 1429716 -Stop resetting border properties for the translation info bar's buttons and menulists. https://reviewboard.mozilla.org/r/212520/#review218220 Looks good, thanks!

(In reply to Dão Gottwald [::dao] from comment #16) > You need to run hg commit like this: > > hg commit -m "Bug 1429716 - Stop resetting border properties for the > translation info bar's buttons and menulists. r=dao" Hello Dao I have pushed the commits.Now what should I do?

Comment on attachment 8942233 [details] Bug 1429716 -Stop resetting border properties for the translation info bar's buttons and menulists. https://reviewboard.mozilla.org/r/212520/#review218220 What should I do now?

(In reply to Ashish Verma [:ashish1500616] from comment #19) > (In reply to Dão Gottwald [::dao] from comment #16) > > You need to run hg commit like this: > > > > hg commit -m "Bug 1429716 - Stop resetting border properties for the > > translation info bar's buttons and menulists. r=dao" > > Hello Dao I have pushed the commits.Now what should I do? I already reviewed your patch, and will push it to autoland repository shortly. Nothing else left to do if all goes well.

Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/785f0ff257ce Stop resetting border properties for the translation info bar's buttons and menulists.r=dao