Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

calcite-list-items are missing focus styling #5880

Closed
ethanbdev opened this issue Dec 2, 2022 · 2 comments
Closed

calcite-list-items are missing focus styling #5880

ethanbdev opened this issue Dec 2, 2022 · 2 comments
Assignees
@geospatialem @benelan
Labels
4 - verified Issues that have been released and confirmed resolved. ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. bug Bug reports for broken functionality. Issues should include a reproduction of the bug.
Milestone
2023 January Prio...

Comments

@ethanbdev
Copy link
Contributor

ethanbdev commented Dec 2, 2022 •
edited by AdelheidF
Loading

Actual Behavior

The list item component seems to be missing the hover styling with the latest version
D7F0BE04-0240-4703-BDDC-39CBC643F5A2

Expected Behavior

There should be some indication of hover/focus on the list item
6517357E-5C95-487B-B079-F38EA2A8D6BB

Reproduction Sample

https://codepen.io/afreitag/pen/MWXzwvp

Reproduction Steps

Hover over a list item, and observe there is no styling

Reproduction Version

next.653

Relevant Info

No response

Regression?

beta.98

Impact

High!

Esri team

ArcGIS Map Viewer
@ethanbdev ethanbdev added bug Bug reports for broken functionality. Issues should include a reproduction of the bug. 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Dec 2, 2022
@github-actions github-actions bot added the ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. label Dec 2, 2022
@AdelheidF AdelheidF mentioned this issue Dec 2, 2022
Closed
driskull added a commit that referenced this issue Dec 5, 2022
@driskull
fix(list-item): Add hover styling. #5880
61e657e
@driskull driskull mentioned this issue Dec 5, 2022
Merged
driskull added a commit that referenced this issue Dec 5, 2022
@driskull
fix(list-item): Add hover styling (#5891)
063d6e9 
**Related Issue:** #5880

## Summary

fix(list-item): Add hover styling


![Dec-05-2022
11-59-25](https://user-images.githubusercontent.com/1231455/205731651-a0ca7383-ae73-4bd3-8c73-717b19d2da64.gif)
@driskull driskull added 3 - installed Issues that have been merged to master branch and are ready for final confirmation. and removed 0 - new New issues that need assignment. labels Dec 5, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2022

Installed and assigned for verification.

@geospatialem geospatialem added 4 - verified Issues that have been released and confirmed resolved. and removed 3 - installed Issues that have been merged to master branch and are ready for final confirmation. needs triage Planning workflow - pending design/dev review. labels Dec 7, 2022
@geospatialem
Copy link
Member

Verified on next.660

verify-list-focus

@AdelheidF AdelheidF reopened this Dec 9, 2022
@geospatialem geospatialem added this to the 2023 January Priorities milestone Dec 9, 2022
@mpayson mpayson mentioned this issue Dec 20, 2022
Closed
@mpayson mpayson mentioned this issue Oct 13, 2023
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geospatialem geospatialem

@benelan benelan

Labels
4 - verified Issues that have been released and confirmed resolved. ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. bug Bug reports for broken functionality. Issues should include a reproduction of the bug.
Projects
None yet
Milestone
2023 January Priorities
Development

No branches or pull requests
5 participants
@driskull @geospatialem @benelan @AdelheidF @ethanbdev