-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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: Rename block on double-click #68744
base: trunk
Are you sure you want to change the base?
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Size Change: +59 B (0%) Total Size: 1.84 MB
ℹ️ View Unchanged
|
Thanks for looking 🙌 This is an alternative PR, with the key difference being that it restricts renaming to list view items that actually have the option to be renamed. It's also using the latest changes in the rename component. That said, we can absolutely move forward with the other PR if preferred. |
Don't have a strong opinion, but I guess we can continue working on the feature here. Why?
Let's make sure we give @rinkalpagdar props when this is merged. |
packages/block-editor/src/components/list-view/block-select-button.js
Outdated
Show resolved
Hide resolved
Thanks, George! Sounds good to me 🫡 |
I want to confirm that the change to what happens when you use the Enter key was removed? Since this affects how you add new paragraphs. |
@carolinan, yes, I believe it was removed in 38c3927. The PR should focus on updating the "double-click" behavior, and any other "shortcuts" for renaming should be discussed separately. |
Hi @carolinan, yes, that was removed so we can revisit and discuss it when deciding on shortcuts and keyboard actions. However, it wasn’t affecting how new paragraphs were added. Pressing “Enter” to rename only worked after a second click in the list view to select the block itself. You can check in trunk that after selecting the block with two clicks, pressing “Enter” doesn’t add a new paragraph. |
Quoting from the issue:
I'm not sure double-clicking is the ideal interaction to rename. The above sounds to me as an assumption that may work for some users but it may be totally unexpected for other users. This kind of interaction patterns should not be introeuced based on personal feeling that it's better in a way or another. They should be user-tested. For me, the new interaction breaks at least one of my typical flows. Instead, with this change, a quick double click would (unexpectedly for me) open the Rename modal dialog.
It may be true that this pattern is used in some very specialized and niche software. However, on the web, a double click on a word just selects that word. Double clicking is mentally associated with the concept of selection since ages and I'm not sure introducing an unexpected behavior is ideal in the first place. |
I also just found this: "Remove Renaming in Pages List Block" #68769 |
Closed my PR (#68546) in favor of this proposal. |
Thanks for the reviews! I appreciate your thoughts and feedback. Just to clarify—renaming for navigation isn't intended. To move forward, I think it’s important to align on one key point: renaming in the list view is currently complicated, especially when users need to rename multiple items, which is a common scenario when creating pages for websites. If we agree on this challenge, we can see there are limited ways to speed up the renaming process—whether through clicks or keyboard shortcuts. Double-click feels like the fastest and most intuitive option. While I understand this can be subjective, many applications I use follow this pattern for renaming, indicating a familiar user experience. Whether it’s triggering a modal or inline renaming (which was previously explored), there needs to be a quick and seamless way to rename items in the list view. Right now, having to right-click, open a menu, and locate “Rename” among 12 options isn’t the best user experience. Not sure what the next steps are if we go with user testing.
It looks like this functionality wouldn't be lost with the changes. You'd just need a slight delay on the second click to maintain that flow.
I understand that concern, though I don't believe this is exclusive to niche software. Single-click feels more naturally tied to selection, while double-click is often associated with other actions. |
Yes and I think the different behavior of a double click depending on the speed will be very confusing for users. I'd also like to remind that double click can be performed only with a pointing device. |
What?
Fixes #66829
Adding the functionality to rename blocks on double click for the list view.
Why?
Renaming a block is currently a bit tricky, requiring several clicks and lacking a keyboard shortcut.
How?
Including the rename modal on double click and adding the ability to rename blocks on double-click.
Testing Instructions
Screenshots or screencast
rename-list-view.mp4