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

Add underline option for text-based blocks #562

Closed
iamthomasbishop opened this issue Feb 7, 2019 · 9 comments · Fixed by #1370
Closed

Add underline option for text-based blocks #562

iamthomasbishop opened this issue Feb 7, 2019 · 9 comments · Fixed by #1370
Assignees
Labels
Aztec Parity Feature exists in Aztec but not yet in Gutenberg mobile [Type] Enhancement Improves a current area of the editor

Comments

@iamthomasbishop
Copy link
Contributor

Context

Core Gutenberg doesn't have an option for underline, but considering the usage data, there is some value in adding it to the style options alongside bold, italic, and strikethrough.

In Aztec, underline is consistently in the top 10, and its usage is roughly double that of strikethrough.

Proposal

Add underline button to Quick Toolbar on text-based blocks such as Heading, Paragraph. I would add this right before the strikethrough option.

screen shot 2019-02-07 at 10 22 28 am

Either of these icons could be used:

@koke
Copy link
Member

koke commented Apr 4, 2019

More on core's decision to not implement underline:

The underline and justify buttons have been removed from the bottom row. Underlining is a bad practice as readers can confuse it with links (bad accessibility), and it does not insert a semantic element. Justifying has uneven browser implementation, and in many cases is bad for readability. Keyboard shortcuts for both will keep working.

And extra discussion in https://core.trac.wordpress.org/ticket/27159

I have mixed feelings about the decision, but I don't think I can stand behind mobile diverging from core in something like this that it's hardly specific to mobile, so I'm closing this for now.

@koke koke closed this as completed Apr 4, 2019
@iamthomasbishop
Copy link
Contributor Author

I also have mixed feelings, but certainly understand the rationale. Closing for now sounds good to me.

@iamthomasbishop
Copy link
Contributor Author

@koke I noticed that core re-added the underline style recently. Do we want to re-open this and consider adding it to mobile?

@koke
Copy link
Member

koke commented Jun 20, 2019

It is there but kind of hidden in an overflow menu.
Screen Shot 2019-06-20 at 22 42 27

I haven't found any discussions as to why the change happened, but I was sold on the original idea of not doing it. I'd be curious to know more about why and how people are using it.

@iamthomasbishop
Copy link
Contributor Author

Same. The rationale of not including it made sense, but perhaps market demand won out 😆 I think it's probably just an expected thing.

@hypest
Copy link
Contributor

hypest commented Aug 5, 2019

Gutenberg on the web still seems to have it nowadays so, re-opening this ticket.

@hypest hypest reopened this Aug 5, 2019
@hypest hypest removed this from the Beta milestone Aug 8, 2019
@hypest hypest added the Aztec Parity Feature exists in Aztec but not yet in Gutenberg mobile label Sep 14, 2019
@SergioEstevao SergioEstevao self-assigned this Sep 19, 2019
@hypest
Copy link
Contributor

hypest commented Oct 4, 2019

Decision on which part to implement for this feature: WordPress/gutenberg#17483 (comment)

@designsimply
Copy link
Contributor

We do still get requests for an underline option for text from apps users. I am not sure yet whether it would be helpful to create a new issue to track such requests, so I will leave the latest request here for now.

The app is great, but it lacks some features that should have become intuitive in the blogging process: 1. Underline under writing […]

4-star review - Sep 18, 2020 at 11:28 PM - Galaxy On Nxt, WPAndroid 15.7-rc-3, Android 7.0
(internal reference: play-store-feedbackid=gp:AOqpTOFwoqUWzFnguYgKjf8J7PtrHbtwz2PdvUIgV_1lrD-OKnxA1b1ZMZt6_sYj9ncV7uZMSlJJ4OaC8x1xG2V6)

I saw in WordPress/gutenberg#17483 that we attempted to add an underline option in WPAndroid at one point but it was removed at the request of the core WordPress & Gutenberg developers. I think what has happened is that it has effectively been removed everywhere except on WordPress.com web. If we were to consider changing this again, or if we want to track requests for it, we might consider adding the option in the app for WordPress.com only which would create feature parity with both WordPress core and WordPress.com.

Mobile Web WordPress.com Mobile Web WordPress Core WPandroid 15.8-rc-1 Aztec
Screenshot_20200921-114252 Screenshot_20200921-114152 Screenshot_20200921-112814

Either way is ok with me, and mainly I would like to know how you would like to receive feedback about this as we continue to receive user requests asking for an underline option through app reviews and elswhere.

@erricgunawan
Copy link

... would like to receive feedback about this as we continue to receive user requests asking for an underline option ...

Just got feedback from 3736130-zen:

Stop disabling keyboard shortcuts such as underlining. Make formatting easier.

I couldn't find where else to report this for now (or if it's still needed to be reported).

Happy to be redirected to the more appropriate place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aztec Parity Feature exists in Aztec but not yet in Gutenberg mobile [Type] Enhancement Improves a current area of the editor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants