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

Update Heading icon #9159

Merged
merged 1 commit into from
Aug 20, 2018
Merged

Update Heading icon #9159

merged 1 commit into from
Aug 20, 2018

Conversation

jasmussen
Copy link
Contributor

This refines the recent heading icon change in #9152 (comment), per feedback.

Although it is H for Heading, internationally that first letter is not quite as ubiquitous, whereas T fares a bit better. By changing from an H we are also avoiding the very same issue that we sought to address by bringing back the Paragraph icon, namely that there are going to be icons (for heading levels) right next to the switcher, and if they look too alike that might cause confusion.

Subheading was also updated to the same, still with the same pattern of a smallcap italicized, to show the relationship between heading and subheading.

screen shot 2018-08-20 at 13 44 07

Note that if a better icon idea comes around, we can always update this again, but for now let's make this the baseline to beat.

This refines the recent heading icon change in #9152 (comment), per feedback.

Although it is H for Heading, internationally that first letter is not quite as ubiquitous, whereas T fares a bit better. By changing from an H we are also avoiding the very same issue that we sought to address by bringing back the Paragraph icon, namely that there are going to be icons (for heading levels) right next to the switcher, and if they look too alike that might cause confusion.
@jasmussen jasmussen added the [Type] Enhancement A suggestion for improvement. label Aug 20, 2018
@jasmussen jasmussen added this to the 3.7 milestone Aug 20, 2018
@jasmussen jasmussen self-assigned this Aug 20, 2018
@jasmussen jasmussen requested a review from mtias August 20, 2018 11:50
@jasmussen jasmussen requested a review from a team August 20, 2018 13:26
@jasmussen jasmussen merged commit f90b2d6 into master Aug 20, 2018
@jasmussen jasmussen deleted the update/heading-icon branch August 20, 2018 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants