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

fix(NcRichContenteditable): disable global allowSpaces tribute option #5321

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Feb 28, 2024

☑️ Resolves

Important

Mentions couldn't be searched with spaces in uid or as a name surname

🖼️ Screenshots

🏚️ Before 🏡 After
image image
image image
image image

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 3️⃣ Backport to next requested with a Vue 3 upgrade

- tributejs library doesn't support allowSpaces as a collection option, only as a global one. As we need it only for mentions ('@' trigger), the best option for now is to disable it completely

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy added 3. to review Waiting for reviews regression Regression of a previous working feature feature: rich-contenteditable Related to the rich-contenteditable components labels Feb 28, 2024
@Antreesy Antreesy added this to the 8.7.2 milestone Feb 28, 2024
@Antreesy Antreesy self-assigned this Feb 28, 2024
Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

@ShGKme time for the fork? 😅

@ShGKme
Copy link
Contributor

ShGKme commented Feb 28, 2024

@ShGKme time for the fork? 😅

Yep

@Antreesy
Copy link
Contributor Author

/backport to next

@Antreesy Antreesy merged commit 781134a into master Feb 28, 2024
18 checks passed
@Antreesy Antreesy deleted the fix/5268/disable-allow-spaces branch February 28, 2024 13:48
@Antreesy Antreesy mentioned this pull request Feb 29, 2024
@ShGKme ShGKme mentioned this pull request Feb 29, 2024
@ShGKme ShGKme modified the milestones: 8.7.2, 8.8.0 Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews feature: rich-contenteditable Related to the rich-contenteditable components regression Regression of a previous working feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants