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

feat: validate CC tag when leaving the field #527

Conversation

BrunoBFerreira
Copy link
Contributor

@BrunoBFerreira BrunoBFerreira commented Oct 2, 2024

Description

When we updated for v4, we changed the behavior of the CC field slightly. Before, if we clicked outside the input box, we would validate the current string and turn it into a tag, when valid. Currently, we are not doing that and we are just saving the input.

Since this feature is useful for our users, we are re-adding this feature in this PR.

JIRA: GG-3804

Screenshots

Need to convert this to a gif

Screen.Recording.2024-10-02.at.16.05.17.mov

Checklist

  • 📗 all commit messages follow the conventional commits standard
  • ⬅️ changes are compatible with RTL direction
  • ♿ Changes to the UI are tested for accessibility and compliant with WCAG 2.1.
  • 📝 changes are tested in Chrome, Firefox, Safari and Edge
  • 📱 changes are responsive and tested in mobile
  • 👍 PR is approved by @zendesk/vikings

@BrunoBFerreira BrunoBFerreira marked this pull request as ready for review October 2, 2024 13:57
@BrunoBFerreira BrunoBFerreira requested a review from a team as a code owner October 2, 2024 13:57
@luis-almeida
Copy link
Contributor

Was the previous behaviour to also keep the focus on the field even after the the blur?

@BrunoBFerreira
Copy link
Contributor Author

Was the previous behaviour to also keep the focus on the field even after the the blur?

Nope. Need to fix that. Thank you @luis-almeida!

@BrunoBFerreira BrunoBFerreira force-pushed the brunobferreira/GG-3804/update-cc-field-behavior-for-v4 branch from 431f25f to 2729a35 Compare October 2, 2024 15:06
@BrunoBFerreira
Copy link
Contributor Author

I believe it should be solved @luis-almeida. Could you take another look? 🙏

@BrunoBFerreira BrunoBFerreira merged commit 872d828 into master Oct 3, 2024
5 checks passed
@BrunoBFerreira BrunoBFerreira deleted the brunobferreira/GG-3804/update-cc-field-behavior-for-v4 branch October 3, 2024 11:52
@zd-svc-github-copenhagen-theme
Copy link
Collaborator

🎉 This PR is included in version 4.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants