Tune design

[iranzo]

This tunes microshift.io design and pages to better show on the browser

[openshift-ci]

Hi @iranzo. Thanks for your PR.

I'm waiting for a redhat-et member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[iranzo]

/assign rootfs

[rootfs]

it looks just some format fixes, are there any material changes?

[iranzo]

Because the "title" already contains this and will show as two h1 elements in the rendered page

[rootfs]

/approve
/ok-to-test

[rootfs]

@iranzo thanks, do you have a preview we can see the final rendering?

[mangelajo]

I have added netlify to build docs/ previews, but not sure how to trigger it on a existing PR.

[mangelajo]

@iranzo can you try to force-push to see if netlify.com will trigger on your PR?

[netlify]

✔️ Deploy Preview for microshift canceled.

🔨 Explore the source changes: 0ad4f8d

🔍 Inspect the deploy log: https://app.netlify.com/sites/microshift/deploys/6176b2093dbcaff4d07cae7a

[iranzo]

Done

[iranzo]

https://deploy-preview-365--trusting-wilson-e71455.netlify.app/

@rootfs @mangelajo

[mangelajo]

/lgtm

[mangelajo]

thanks @iranzo it looks great

[iranzo]

thanks @iranzo it looks great

Thanks, it's just some basic tuning to get it looking better on the rendered site, but using a stock theme + admonitions (check the 'note' in the index page for example) and some other stuff for SEO (keywords, sitemap, twitter cards, etc)

If you later get OSPO to design a UI there will be other stuff to request :)

[iranzo]

I have added netlify to build docs/ previews, but not sure how to trigger it on a existing PR.

BTW you can tune the domain used so instead of 'trusting-wilson' it says something like microshif on netlify site, for example in kubevirt: https://deploy-preview-807--kubevirtio.netlify.app/

[mangelajo]

done, I guess it will be effective on next push, https://microshift.netlify.app/

[mangelajo]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mangelajo, rootfs

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [mangelajo,rootfs]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[iranzo]

Yup, actual URL: https://deploy-preview-365--microshift.netlify.app/

[mangelajo]

yup it seems to work, canceled because: 3:33:12 PM: Failed during stage 'checking build content for changes': Canceled build due to no content change 3:33:12 PM: Finished processing build request in 11.997644152s