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

List View: Add ability to remove items #30109

Open
annezazu opened this issue Mar 22, 2021 · 3 comments
Open

List View: Add ability to remove items #30109

annezazu opened this issue Mar 22, 2021 · 3 comments

Comments

@annezazu
Copy link
Contributor

@annezazu annezazu commented Mar 22, 2021

What problem does this address?

As part of the third call for testing with the FSE Outreach Program, this came up in the feedback:

Needs power features including a re-order and delete feature for each block.

This was previously discussed here a few years ago #10984 but List View was determined to be a bit too new at the time to be explored further. It was then subsequently closed out.

This might be relevant for @priethor as I see you're gathering a list of improvements here: #29733

What is your proposed solution?

Allow the ability to remove items from the List View. When considering this, I do want to bring in feedback from the previous discussion from @afercia:

Multiple actions on a single UI controls are (maybe) a pattern best suited for mobile applications. On the Web, good semantics is key. An Ui control should have only one, clear, available action. From an accessibility perspective, I'm not opposed to explore extending the functionalities available in the Block Navigation menu, but I'd strongly recommend a separate control (another button) to delete a block. Also, when deleting an item, focus must be managed programmatically, as the previously focused element gets removed and focus must land in the most logical place.

@talldan
Copy link
Contributor

@talldan talldan commented Mar 25, 2021

The block more menu is implemented in list view but inactive currently and that allows deletion - #22427

When I looked at adding this to the post editor, the accessibility team were concerned about the complexity of grid navigation in List View for users, which is needed if we place a more menu or delete button in the row of the list view item: #25034 (comment)

@annezazu
Copy link
Contributor Author

@annezazu annezazu commented Mar 25, 2021

Ah ha! Thank you for sharing the wider context and concerns around accessibility 🙌🏼

@carolinan
Copy link
Contributor

@carolinan carolinan commented Apr 12, 2021

This was also mentioned during user testing on April 9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants