Remove obsolete .focusring styling

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P3
normal
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: dao, Assigned: devikasugathan007, Mentored)

Tracking

({good-first-bug})

Trunk
mozilla63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

(Whiteboard: [lang=css])

Attachments

(1 attachment, 3 obsolete attachments)

Reporter

Description

9 months ago
https://searchfox.org/mozilla-central/search?q=.focusring&case=true&regexp=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.

Comment 1

9 months ago
Can I work on this issue??
Reporter

Comment 2

9 months ago
(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.

Comment 3

9 months ago
You are right!! 
I have already worked on this type of bugs.

I will try to work on some more advance bugs.
Assignee

Comment 4

9 months ago
Can I work on this bug?
Reporter

Comment 5

9 months ago
(In reply to Devika Sugathan from comment #4)
> Can I work on this bug?

Sure!
Assignee

Comment 6

9 months ago
Hi, I just to delete those lines right? Please correct me if I'm wrong.
Assignee

Comment 7

9 months ago
Posted patch Remove.patch (obsolete) — Splinter Review
Attachment #9002266 - Flags: review?(dao+bmo)

Comment 8

9 months ago
You also need to remove the lines in toolkit/themes/shared/tree.inc.css
Reporter

Comment 9

9 months ago
Comment on attachment 9002266 [details] [diff] [review]
Remove.patch

What Tim said...
Attachment #9002266 - Flags: review?(dao+bmo)
Assignee

Comment 10

9 months ago
Posted image Screenshot_20180822_195614.png (obsolete) —
I could not find the file in toolkit/themes/shared/tree.inc.css.
Attachment #9003142 - Flags: feedback?
Reporter

Comment 11

9 months ago
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.
Attachment #9003142 - Flags: feedback?
Assignee

Comment 12

9 months ago
I have rebased it but still, I couldn't find it.

Comment 13

9 months ago
(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` ?
Assignee

Comment 14

9 months ago
Posted image Screenshot_20180822_220955.png (obsolete) —

Comment 15

9 months ago
(In reply to Devika Sugathan from comment #14)
> Created attachment 9003197 [details]
> Screenshot_20180822_220955.png

Try `hg pull` then `hg rebase -d central`
Assignee

Comment 16

9 months ago
Posted patch Added.patchSplinter Review
Attachment #9003142 - Attachment is obsolete: true
Attachment #9003197 - Attachment is obsolete: true
Attachment #9003225 - Flags: review?(dao+bmo)

Updated

9 months ago
Attachment #9002266 - Attachment is obsolete: true

Updated

9 months ago
Assignee: nobody → devikasugathan007
Reporter

Comment 17

9 months ago
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+
Reporter

Updated

9 months ago
Keywords: checkin-needed

Comment 18

9 months ago
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ccaeab2823a2
Remove obsolete .focusring styling r=dao
Keywords: checkin-needed
Comment hidden (obsolete)

Comment 20

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ccaeab2823a2
Status: NEW → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Reporter

Updated

9 months ago
Flags: needinfo?(dao+bmo)
You need to log in before you can comment on or make changes to this bug.