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

Highlight curly apostrophe #248

Merged
merged 1 commit into from
Apr 20, 2020
Merged

Highlight curly apostrophe #248

merged 1 commit into from
Apr 20, 2020

Conversation

sebastienserre
Copy link
Contributor

Highlight when a curly apostophe is used

@audrasjb audrasjb added this to the 1.6 milestone Apr 19, 2020
@audrasjb audrasjb self-requested a review April 19, 2020 09:52
@audrasjb
Copy link
Collaborator

Hey, thanks for the enhancement.
Looks good to me, I only wonder if we want to check !gd_get_setting('curly_apostrophe_warning') or gd_get_setting('curly_apostrophe_warning') 🤔

Copy link
Collaborator

@audrasjb audrasjb left a comment

Choose a reason for hiding this comment

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

Approved.

@Mte90
Copy link
Owner

Mte90 commented Apr 20, 2020

Can I have a translation string/page where I can test it before merge and do a new release?

@sebastienserre
Copy link
Contributor Author

@Mte90 Mte90 self-requested a review April 20, 2020 13:39
Copy link
Owner

@Mte90 Mte90 left a comment

Choose a reason for hiding this comment

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

Tested and works!
Just for the future if you can update also the changelog, I will do it for this case :-D

@Mte90 Mte90 merged commit 6bcd411 into Mte90:master Apr 20, 2020
@garretthyder
Copy link
Collaborator

Nice, taking a glance I wonder if we highlight all 'curly' quotes (single/double right/left)? This PR currently only does the apostrophe (single right quote). Just a thought while the topic is active.

@Mte90
Copy link
Owner

Mte90 commented Apr 21, 2020

Just because we are talking about highlighting symbols, if we do a text field in the settings where the user inserts the symbols that want highlighted?
In this way we can support more symbols and the users can insert the ones prefer.

@garretthyder
Copy link
Collaborator

I like that, we can default to pre-generated characters and allow users to disable or customize the highlighted characters.

@Mte90
Copy link
Owner

Mte90 commented Apr 22, 2020

Created an issue for that :-D

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

Successfully merging this pull request may close these issues.

None yet

4 participants