-
Notifications
You must be signed in to change notification settings - Fork 716
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
What a drag #12396
What a drag #12396
Conversation
Build Artifacts
|
@radinamatic - noting that there are some underlying problems here with aria labeling and overall screenreader (vs. sighted keyboard user) functionality. and we have some refactoring to do in general with the drag and drop feature. i'll be filing follow up issues for this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes all make sense, although the composable being added to the core apiSpec seems unnecessary at this point :)
Tested in both Firefox and Chrome in Ubuntu 20.04
This works correctly when the user added the titles to the sections, but fails when the sections have the default names: Default section titles drag-default-section-titlesmp4.mp4Custom section titles drag-custom-section-titles.mp4
|
…their default title numbers even when reordering, until the user Applies Settings
…re that the active tab persists as active, even if it changes positions
effdce0
to
2ae41d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to be working according to my expectations, although the mouse driven drag and drop appears to have some rough edges to it still.
Summary
This adds a very tiny
useDrag
composable (which I, with great restraint, did not fill with pop culture drag references) and uses this + some additional functions to ensure that the keyboard accessibility of the drag sort order worksFixes the draggable items part of #12148
References
Reviewer guidance
There are two sections that should be fixed by this
Creating quiz questions, and editing section order
Testing checklist
PR process
Reviewer checklist
yarn
andpip
)