Skip to content

added optional gradient. added halo on active. added aria labels to a…#714

Open
GregLongo wants to merge 1 commit intoinfluxdata:masterfrom
GregLongo:gl_interview
Open

added optional gradient. added halo on active. added aria labels to a…#714
GregLongo wants to merge 1 commit intoinfluxdata:masterfrom
GregLongo:gl_interview

Conversation

@GregLongo
Copy link

…ll inputs

Closes #

---Note: This is an interview assignment for UX Engineer Position --

Changes

Modified Range Slider component :

  • Added subtle gradient effect with toggle in storybook
  • Added halo effect around slider handle on active state for greater visibility on mobile and behind desktop cursor
  • Added aria-labels on inputs for screen readers

Screenshots

// Add screenshots here if relevant

Checklist

Check all that apply

  • Updated documentation to reflect changes
  • Added entry to top of Changelog with link to PR (not issue)
  • Tests pass
  • Peer reviewed and approved
  • [ x] Signed CLA (if not already signed)

@GregLongo GregLongo requested review from a team, juliajames12 and mavarius as code owners January 8, 2022 23:43
Comment on lines +31 to +33
// &:focus {
// outline: none;
// }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused code should be removed not commented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants