Closed Bug 1511152 Opened 6 years ago Closed 6 years ago

[meta] Eslint coverage for JSX a11y

Categories

(Firefox :: Messaging System, task, P1)

Product:
Firefox
Component:
Messaging System
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox65 --- affected

People

(Reporter: yzen, Assigned: emcminn)

References

(Depends on 1 open bug, Blocks 2 open bugs)

Details

(Keywords: access, meta)

Attachments

(1 file)

Link to GitHub pull-request: https://github.com/mozilla/activity-stream/pull/4583
52 bytes, text/x-github-pull-request
Details | Review
This could be an easy win for accessibility as it can flag some low hanging fruit problems. The plugin repo is here: https://github.com/evcohen/eslint-plugin-jsx-a11y
Hey kate, can you take at look at this one, please?
Flags: needinfo?(khudson)
Blocks: 1548268

Added comments in the pull-request to see what is still outstanding to be able to merge this. So far this has helped us a lot with trailhead-related code

Type: enhancement → task
Flags: needinfo?(khudson)

I'm updating this to a meta to track outstanding work around enabling this for all new tab content

No longer blocks: 1548268
Depends on: 1548268
Keywords: meta
Summary: Consider using eslint plugin for testing JSX a11y. → Use eslint plugin for testing JSX a11y on New Tab
Summary: Use eslint plugin for testing JSX a11y on New Tab → Eslint coverage for JSX a11y
Depends on: 1551566
Summary: Eslint coverage for JSX a11y → [meta] Eslint coverage for JSX a11y
Depends on: 1551613
Assignee: yzenevich → emcminn
Depends on: 1551711
Priority: -- → P1
No longer depends on: 1551711
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Depends on: 1554739
Component: Activity Streams: Newtab → Messaging System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: