Closed Bug 1483810 Opened 3 years ago Closed 3 years ago
Remove obsolete .focusring styling
https://searchfox.org/mozilla-central/search?q=.focusring&case=true®exp=false&path=css This seems to be unused in mozilla-central. There are some uses in comm-central but they can add custom CSS to keep supporting this if needed.
Can I work on this issue??
(In reply to Sahil Bhosale from comment #1) > Can I work on this issue?? I don't think there's much you can learn here anymore, this is very similar to other good first bugs that you worked on.
You are right!! I have already worked on this type of bugs. I will try to work on some more advance bugs.
Can I work on this bug?
(In reply to Devika Sugathan from comment #4) > Can I work on this bug? Sure!
Hi, I just to delete those lines right? Please correct me if I'm wrong.
You also need to remove the lines in toolkit/themes/shared/tree.inc.css
Comment on attachment 9002266 [details] [diff] [review] Remove.patch What Tim said...
I could not find the file in toolkit/themes/shared/tree.inc.css.
Comment on attachment 9003142 [details] Screenshot_20180822_195614.png Your mozilla-central copy is probably outdated, so you'll have to update it with hg pull and hg update. Make sure you use hg pull --rebase if you have local commits, or remove your commits before updating.
I have rebased it but still, I couldn't find it.
(In reply to Devika Sugathan from comment #12) > I have rebased it but still, I couldn't find it. what's the output of your `hg wip` ?
(In reply to Devika Sugathan from comment #14) > Created attachment 9003197 [details] > Screenshot_20180822_220955.png Try `hg pull` then `hg rebase -d central`
Comment on attachment 9003225 [details] [diff] [review] Added.patch Looks good, except for the commit message: >Bug 1483810 Remove obsolete .focusring styling *** Changed better: Bug 1483810 - Remove obsolete .focusring styling. r=dao
Attachment #9003225 - Flags: review?(dao+bmo) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/ccaeab2823a2 Remove obsolete .focusring styling r=dao
You need to log in before you can comment on or make changes to this bug.