hlxsites / elixirsolutions

Apache License 2.0
0 stars 1 forks source link

wrap-text #243

Closed sforschen closed 10 months ago

sforschen commented 10 months ago

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #242

Test URLs:

aem-code-sync[bot] commented 10 months ago

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed. In case there are problems, just click a checkbox below to rerun the respective action.

Commits * [0d5eba9](https://github.com/hlxsites/elixirsolutions/pull/243/commits/0d5eba96e347be9f919ea347ee40c17620bfb940) :white_check_mark: (latest)
aem-code-sync[bot] commented 10 months ago
Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
sforschen commented 10 months ago

@janefrommaine I was watching a CSS video about text and was like I bet this will make the text on the site look better.

I tested it this morning, and it just cleans up the line breaks to a much nicer cadence.

sforschen commented 10 months ago

@janefrommaine ah yeah pretty is a little more subtle. It for the most part cleans up orphans in the body copy. While it doesn't have as much support it helps us author content that is easier to read and looks better. I often have Stephanie rewrite content if it leaves an orphan and looks bad. It also helps on mobile. The vast majority of our traffic is Chrome and Edge. On the about us page: image Weightloss blog: image