cncf / cncf.io

☁️♮🏛🚧 The CNCF.io WordPress website
https://cncf.io
MIT License
80 stars 37 forks source link

Launch CTO Summit NA 2022 Report #663

Closed thetwopct closed 1 year ago

thetwopct commented 1 year ago

Workflow issue

Current PR

PR page for review https://pr-670-cncfci.pantheonsite.io/reports/cto-summit-na-2022/

Checklist:

IMAGES

LINKS

TEXT

CONFORMANCE

GO LIVE

GATED

GA

thetwopct commented 1 year ago

The PR has a few things to fix:

1) The event was in October 2022, so guessing the copyright notice might need to update?

image

2) Badly capitalised words

image

3) On mobile lack of spacing here:

image

4) Lack of spacing under See More:

image

5) Long word pushes out screen:

image

6) The CNCF logo has some white space around it, on mobile, this makes it misaligned.

7) Fix incorrect character encoding

Screenshot-2023-02-28 --16 49 49@2x

cjyabraham commented 1 year ago

A few more:

  1. There should be no bottom line below Priyanka on the CTO SUMMIT COMMITTEE LIST
  2. Spacing above quote mark is too large
  3. Quote icons are smaller than they should be
  4. Quotes should not have " around them in addition to the styling
  5. LF Research logo at the bottom should be SVG
thetwopct commented 1 year ago

OK have pushed updates: