CUCentralAdvancement / essential-cu

Home for content related to https://essential.cu.edu/.
https://essential.cu.edu/
MIT License
1 stars 0 forks source link

Home page edits #471

Closed tskillern closed 2 years ago

tskillern commented 2 years ago
Feissit commented 2 years ago

I have made these changes and created a pull request. @alexfinnarn can you reply here when the changes are merged so Tim can validate? Thank you!

alexfinnarn commented 2 years ago

That pull request has been merged.

tskillern commented 2 years ago

I think if there’s a hard return and “fueling” is capped, that’d work.

From: Feissit @.> Date: Tuesday, November 2, 2021 at 2:04 PM To: CUCentralAdvancement/essential-cu @.> Cc: Tim Skillern @.>, Mention @.> Subject: Re: [CUCentralAdvancement/essential-cu] Home page edits (Issue #471)

@tskillernhttps://github.com/tskillern -- I changed this to two sentences with periods to break it up, but I don't think it looks as nice/clean as

Philanthropy is transformative Fueling passion, motivation and joy

It's not grammatically correct, but the bolded text reads pretty clear imo. Can we revert to the previous version? A dash might look odd here too

[Screen Shot 2021-11-02 at 2 02 20 PM]https://user-images.githubusercontent.com/91230278/139943891-e435bef6-6b33-4089-bef2-504dc1519b6c.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/CUCentralAdvancement/essential-cu/issues/471#issuecomment-958125515, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQWLOG7NZP4LZOY2JYDJ2TTUKBG5NANCNFSM5HGX4Z4A. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

tskillern commented 2 years ago

All home page content edits are confirmed and good to go. @alexfinnarn @Feissit