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.

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

Listview: Readonly / Non-selectable listview should not have hover / selected styling #4452

Closed
volante007 opened this issue Oct 5, 2020 · 1 comment · Fixed by #4494
Closed
Assignees
Labels
team: homepages Issues for the homepages team type: bug 🐛 [1] Velocity rating (Fibonacci)

Comments

@volante007
Copy link

volante007 commented Oct 5, 2020

Describe the bug
A listview with selectable: false still gets focus / hover styles.

To Reproduce

  1. Go to https://design.infor.com/code/ids-enterprise/latest/demo/components/listview/example-index?theme=uplift&variant=light&colors=0066D4
  2. Hover the items
  3. Select an item

list

Expected behavior
I think a readonly / non-selectable list should just be a list of items, no special hover / selected styling. The items are just info.
There are of course scenarios where a non-selectable listview should have some kind of hover style, e.g. list of links / clickable (but not selectable) items, but I think a "plain" list should be the default, especially since these styles are difficult to override by the consumer. Alternatively an option that makes the list "plain", to avoid having to override focus / hover / active etc for info-only lists.

Version

  • ids-enterprise: v4.32.0
@tmcconechy tmcconechy added [1] Velocity rating (Fibonacci) type: bug 🐛 labels Oct 5, 2020
@tmcconechy tmcconechy added this to To do in Enterprise 4.34.x (Oct 2020) Sprint via automation Oct 5, 2020
@tmcconechy tmcconechy added the team: homepages Issues for the homepages team label Oct 5, 2020
@ericangeles ericangeles self-assigned this Oct 14, 2020
@ericangeles ericangeles moved this from To do to In progress in Enterprise 4.34.x (Oct 2020) Sprint Oct 14, 2020
@ericangeles ericangeles moved this from In progress to Pending Review in Enterprise 4.34.x (Oct 2020) Sprint Oct 14, 2020
@tmcconechy tmcconechy moved this from Pending Review to Ready for QA (beta) in Enterprise 4.34.x (Oct 2020) Sprint Oct 19, 2020
@janahintal
Copy link
Contributor

This issue is now resolved. Thank you.
image

@janahintal janahintal moved this from Ready for QA (beta) to Done in Enterprise 4.34.x (Oct 2020) Sprint Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team: homepages Issues for the homepages team type: bug 🐛 [1] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants