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

[Bug]: Punctuation marks included in hashtag parsing blue text color #922

Closed
nekohayo opened this issue Apr 21, 2024 · 2 comments · Fixed by #923
Closed

[Bug]: Punctuation marks included in hashtag parsing blue text color #922

nekohayo opened this issue Apr 21, 2024 · 2 comments · Fixed by #923
Labels
bug Something isn't working

Comments

@nekohayo
Copy link

Describe the bug

Filing it separately from #920 as I presume it might be a separate parsing issue:

Punctuation marks like . , ; : ' are included in the blue formatting of hashtags. It seems like it might just be a visual glitch, and does not end up being "part of the tag" once posted.

Steps To Reproduce

Type Beware this #hashtag, for its blue ink leaks onto the comma!

Logs and/or Screenshots

image

I have not tested posting to verify if the straight apostrophe ' is a valid hashtag character for Mastodon or not, as it would be reminiscent of #875 and #883, but I wonder if this is normal too:

image

Instance Backend

Mastodon

Operating System

Fedora 39

Package

Flatpak

Troubleshooting information

os: GNOME 46 (Flatpak runtime)
prefix: /app
flatpak: true
version: 0.7.2 (production)
gtk: 4.14.1 (4.14.1)
libadwaita: 1.5.0 (1.5.0)
libsoup: 3.4.4 (3.4.4)
libgtksourceview: 5.12.0 (5.12.0)

Additional Context

No response

@GeopJr
Copy link
Owner

GeopJr commented Apr 21, 2024

The regex was too loose, it should now only allow: any alphabetic character of any language, any number, _

@nekohayo
Copy link
Author

This should also be targetted to 0.8.x :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants