Skip to content

Conversation

@KHVBui
Copy link
Collaborator

@KHVBui KHVBui commented Aug 1, 2022

  • Fix linter errors to allow regular builds going forward
  • Replaced '==' with '===' to be typesafe
  • Created temporary "alt" prop for images
  • Resolved "no-static-element-interactions" by giving the "button" role to the mobile menu element

KHVBui added 4 commits August 1, 2022 12:44
- Type-safe equality operators are preferred
- A lone div with "onClick" must have some role to add interactivity
- Buttons must be focusable by having a tabIndex that lets users reach the element by pressing tab
@KHVBui KHVBui requested a review from minhngo3818 August 1, 2022 20:16
@KHVBui KHVBui self-assigned this Aug 1, 2022
@KHVBui KHVBui added Resources-page resouces page Projects-page projects-page fix fix bug Nav-Bar navigation bar Cleanup Deleting files, comments, and unnecessary code labels Aug 1, 2022
@minhngo3818 minhngo3818 merged commit cae51a2 into main Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cleanup Deleting files, comments, and unnecessary code fix fix bug Nav-Bar navigation bar Projects-page projects-page Resources-page resouces page

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants