Skip to content

Use TagsView to manage post tags #24670

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

Draft
wants to merge 3 commits into
base: trunk
Choose a base branch
from

Conversation

crazytonyli
Copy link
Contributor

Note

This PR is built on top of #24668.

Description

See the recording below.

add-tags.mp4

Testing instructions

@dangermattic
Copy link
Collaborator

1 Warning
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.
1 Message
📖 This PR is still a Draft: some checks will be skipped.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jul 14, 2025

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number28219
VersionPR #24670
Bundle IDcom.jetpack.alpha
Commit09bf586
Installation URL25ane7noicsjg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jul 14, 2025

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number28219
VersionPR #24670
Bundle IDorg.wordpress.alpha
Commit09bf586
Installation URL5rup4tk9h8sig
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@kean
Copy link
Contributor

kean commented Jul 14, 2025

Hey, I was planning to rework it in the scope of https://linear.app/a8c/project/post-settings-and-publishing-next-steps-ecf77520ab3c. It's great that you decided to address it, but I think it needs to largely retain the current UX where you enter the tags using a text field that automatically becomes first responder when you open the screen and uses auto-complete to add existing tags. You should also be able to add new tags from this screen by simply adding them to the text field.

I think the only option how we could improve the current UX is to add token fields to the text field to make it easier to remove the tags you don't want.

Base automatically changed from feature/tags-data-view to trunk July 15, 2025 22:12
@crazytonyli
Copy link
Contributor Author

@kean Oh nice. I didn't notice that project. I started working on this because I'd like to fix this bug on searching tags.

You should also be able to add new tags from this screen by simply adding them to the text field.

I think the only option how we could improve the current UX is to add token fields

Good catch! I missed adding new tags.

Would you like to take this PR over and make it part of the post settings project, or would you prefer me to iterate on this PR? I'm okay either way, just don't want to step on your toes.

@crazytonyli crazytonyli force-pushed the feature/tags-data-view-post-settings branch from 303aae9 to 09bf586 Compare July 20, 2025 23:48
Copy link

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

Successfully merging this pull request may close these issues.

4 participants