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

Dynamic matrix expand button has wrong focus. #7595

Closed
mikeamelin opened this issue Dec 28, 2023 · 0 comments · Fixed by #7604
Closed

Dynamic matrix expand button has wrong focus. #7595

mikeamelin opened this issue Dec 28, 2023 · 0 comments · Fixed by #7604
Assignees
@novikov82
Labels
bug
Milestone
v1.9.125

Comments

@mikeamelin
Copy link

mikeamelin commented Dec 28, 2023 •
edited
Loading

In current moment we have no special state for focusing, so we should use the same as hovering —
https://www.figma.com/file/i6Lpc7zkjHtso2oXZV97fo/*-Components?type=design&node-id=2658-40078&mode=design&t=H9JnLms38sPXQEjn-4
Image
@mikeamelin mikeamelin self-assigned this Dec 28, 2023
@mikeamelin mikeamelin added the bug label Dec 28, 2023
novikov82 added a commit that referenced this issue Dec 28, 2023
@novikov82
#7595 Dynamic matrix expand button has wrong focus.
b9a1541 
Fixes #7595
@novikov82 novikov82 mentioned this issue Dec 28, 2023
Merged
novikov82 added a commit that referenced this issue Jan 9, 2024
@novikov82
#7595 Dynamic matrix expand button has wrong focus. - reset focus in …
1ea11df 
…tests

Fixes #7595
OlgaLarina added a commit that referenced this issue Jan 9, 2024
@OlgaLarina
Merge pull request #7604 from surveyjs/issue/7595-Dynamic-matrix-expa…
f780ff6 
…nd-button-has-wrong-focus

#7595 Dynamic matrix expand button has wrong focus.
@mikeamelin mikeamelin removed their assignment Jan 9, 2024
@OlgaLarina OlgaLarina added this to the v1.9.125 milestone Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@novikov82 novikov82

Labels
bug
Projects
None yet
Milestone
v1.9.125
Development

Successfully merging a pull request may close this issue.

#7595 Dynamic matrix expand button has wrong focus. surveyjs/survey-library
3 participants
@OlgaLarina @mikeamelin @novikov82