Comments

Sebacic created an issue. See original summary.

andrewmacpherson’s picture

I agree. Headings are an arbitrary length, and will often be as long as an entire sentence. All-caps will make it harder to read, and users with dyslexia more severely.

Adding a related issue which discusses the the problem in existing core themes. It has lots more links to various style guides and articles which take a dim view of all-caps.

ellenoise’s picture

StatusFileSize
new875.16 KB

This issue would be very easy to fix. In `src/css/components/main.css`, remove Line 38: `text-transform: uppercase;`. Here's what the headings look like when the H2 uppercase styling is removed:

proeung’s picture

Assigned: Sebacic » jwitkowski79

I'd like to get @jwitkowski79's input on this as well. What do you think about dropping all the uppercase for the H2 heading?

jwitkowski79’s picture

@proeung: I think we're okay dropping the uppercase for H2. I just did a small test on what this would look like and there's enough of a difference in the font size and weight that it still reads as an H2

jwitkowski79’s picture

Assigned: jwitkowski79 » Unassigned
andrewmacpherson’s picture

That's a nice outcome then. Let's go for it.

thejimbirch’s picture

Component: Proof of Concept » Code
Category: Feature request » Bug report
Status: Active » Needs review
StatusFileSize
new393 bytes

Patch attached that removes text-transform: uppercase on H2s.

mherchel’s picture

Status: Needs review » Needs work

@thejimbirch can you re-roll this? Note you'll need to attach both the src (in css/src/) and distribution (in css/dist/).

kostyashupenko’s picture

Status: Needs work » Needs review
StatusFileSize
new720 bytes
mherchel’s picture

Status: Needs review » Fixed

Committed!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.