Skip to content

Conversation

@kmanijak
Copy link
Contributor

@kmanijak kmanijak commented Sep 15, 2025

What?

Closes

Combine Order and Order By control into single dropdown. cc: @mikachan

Why?

Dropdowns occupy quite a lot of space in Inspector Controls and are pretty heavy. There's no need for separate dropdown for Order and OrderBy while it can be easily combined. Agreed in this comment.

I also removed order by slug as I can't really see a use case for it and if so, I treat it a UI pollution.

How?

Testing Instructions

  1. Ensure experimental blocks are enabled.
  2. Insert the Terms Query block in a template.
  3. Make sure the default order is Name: A → Z
  4. Change the ordering and make sure it's reflected in Editor

Testing Instructions for Keyboard

Screenshots or screencast

Before After
image image image

@github-actions
Copy link

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 props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: kmanijak <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@mikachan mikachan self-requested a review September 17, 2025 08:52
@mikachan mikachan added [Type] Enhancement A suggestion for improvement. [Block] Terms Query Affects the Terms Query Block labels Sep 17, 2025
Copy link
Member

@mikachan mikachan left a comment

Choose a reason for hiding this comment

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

I also removed order by slug as I can't really see a use case for it and if so, I treat it a UI pollution.

This makes sense; makes the UI look cleaner.

This works as described, and the code looks good, great job! 🚢

@mikachan mikachan merged commit b8972ed into WordPress:trunk Sep 17, 2025
75 of 78 checks passed
@github-actions github-actions bot added this to the Gutenberg 21.7 milestone Sep 17, 2025
@kmanijak
Copy link
Contributor Author

Thanks @mikachan, I'll rebase #71633 soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Terms Query Affects the Terms Query Block [Type] Enhancement A suggestion for improvement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants