-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: tiny layout improvements #54
Conversation
Signed-off-by: thisisobate <[email protected]>
✅ Deploy Preview for konveyor-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for glowing-cassata-a7b83f ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Signed-off-by: thisisobate <[email protected]>
I took a look. That paragraph text in the header is still hard to read. Where as sub heading text is much easier to read on this web page (different website.) Can a bit bigger and reduce line height to accommodate for this? Sorry for the tweaks. But it does affect legibility a lot. Font size: 1.125 REM, 1.5 line height. We can try applying this to all paragraph text. Feel free to suggest other solutions to make text legible. |
Signed-off-by: thisisobate <[email protected]>
Fixed...feel free to take a look @JonathanR19 |
Signed-off-by: thisisobate <[email protected]>
Signed-off-by: thisisobate <[email protected]>
Signed-off-by: thisisobate <[email protected]>
Signed-off-by: thisisobate <[email protected]>
fixes #52 #51