-
Notifications
You must be signed in to change notification settings - Fork 87
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
Bump docsy theme to latest version 0.9.1 #216
Conversation
✅ Deploy Preview for bejewelled-pegasus-b0ce81 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hi @deining , You're welcome at the meeting Check to community calendar for more details of meetings at Layers. |
Thanks for your invitation! Due to other commitments, I'm unable to attend. Please use this issue tracker if any further input is needed from my side. Looking forward receiving your review of my PRs! |
Thanks for sending the word @deining |
Thanks! |
@deining what issues did you see upon upgrade (if any at all)? |
Fortunately, I didn't see any issues when upgrading 😄 . |
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.
Thank you for working on this @deining
I noticed some texts (on the home page) use a font different from the Layer5 brand font. You may want to take a look at that.
Signed-off-by: Andreas Deininger <[email protected]>
75f2673
to
c343853
Compare
Signed-off-by: Andreas Deininger <[email protected]>
fc76f78
to
ddd30ae
Compare
You are welcome.
With an additional commit, I tried to resolve this. Still not 100% the original layout, though. |
Looks good now. Thank you @deining |
@iArchitSharma how are we proceeding here? |
Fixing this in PR - #230 |
Notes for Reviewers
This PR bumps docsy theme to latest released version 0.9.1.
Signed commits