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

Fix: change hover scroll behaviour #1043

Merged
merged 2 commits into from
Aug 3, 2021
Merged

Conversation

thewahome
Copy link
Collaborator

Overview

Enhances #1023 to use CSS instead of javascript functions to show the hover

@thewahome thewahome changed the title Fix? change hover scroll behaviour Fix: change hover scroll behaviour Aug 3, 2021
onItemInvoked={this.querySelected}
/>
</div>
<DetailsList
Copy link
Contributor

Choose a reason for hiding this comment

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

We need the div with the navigation role for accessibility purposes

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

aaah I see

@thewahome thewahome merged commit a030a20 into dev Aug 3, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Aug 3, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-1043.centralus.azurestaticapps.net

thewahome added a commit that referenced this pull request Aug 3, 2021
* Task: Add chat-id & membership-id (#1024)

* Fix: query link parsing with trailing '.' (#1033)

* Feature: DetailsList scrollbars only appear onHover (#1023)

* Feature: automatic dark light mode (#1019)

* Task: add translated aria labels (#1040)

* Task: remove sandbox api test (#1042)

* Fix: change hover scroll behaviour (#1043)

* Fix: Accessibility bugs - Aug  (#1041)

* Feature: Add Expanding Query Input Field (#1020)
@thewahome thewahome deleted the fix/change-hover-scroll-behaviour branch August 6, 2021 13:17
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