Closed Bug 1551613 Opened 6 months ago Closed 6 months ago

Enable jsx-a11y eslint for TopSites component and fix errors

Categories

(Firefox :: Messaging System, task, P1)

task

Tracking

()

RESOLVED FIXED
Firefox 68
Iteration:
68.4 - Apr 29 - May 12
Tracking Status
firefox68 --- fixed

People

(Reporter: k88hudson, Assigned: emcminn)

References

Details

(Keywords: github-merged)

Attachments

(1 file)

We added eslint jsx-a11y coverage for Trailhead files in https://bugzilla.mozilla.org/show_bug.cgi?id=1548268; let's extend that coverage to to TopSites component and fix any errors that come up.

This is the line that currently configures coverage: https://github.com/mozilla/activity-stream/blob/84543b06c76c16778acbde1110288cde7c56e566/yamscripts.yml#L67

Type: defect → task
Assignee: nobody → emcminn
Blocks: 1552599
Iteration: --- → 69.1 - May 13 - 26
Status: NEW → RESOLVED
Closed: 6 months ago
Keywords: github-merged
Priority: -- → P1
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
Iteration: 69.1 - May 13 - 26 → 68.4 - Apr 29 - May 12
Component: Activity Streams: Newtab → Messaging System
You need to log in before you can comment on or make changes to this bug.