-
Notifications
You must be signed in to change notification settings - Fork 578
Migrate docsy 0.13.0 #3608
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
base: main
Are you sure you want to change the base?
Migrate docsy 0.13.0 #3608
Conversation
✅ Deploy Preview for cncfglossary ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
ff9d728 to
ad7f3f2
Compare
Signed-off-by: Junya Okabe <[email protected]> Signed-off-by: Junya Okabe <[email protected]>
5eaa72c to
afa6f99
Compare
cc8b917 to
853b971
Compare
Signed-off-by: Junya Okabe <[email protected]>
853b971 to
82f822e
Compare
jihoon-seo
left a comment
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.
It seems that the preview works nice.
@Okabe-Junya Thank you for this huge work!
cjyabraham
left a comment
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.
There are some significant shifts in the layout and styles of the site. I just opened the homepage and can see about 6 things that are wrong. Is this meant to be reviewed at this stage or is it still a WIP?
cjyabraham
left a comment
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.
There are some significant shifts in the layout and styles of the site. I just opened the homepage and can see about 6 things that are wrong. Is this meant to be reviewed at this stage or is it still a WIP?
|
I think the new "Create child page" and "View page source" links are ok to leave but fine to remove as well if we want to keep the list as simple as possible. As for the render issues, here is an initial list. I think they all deserve fixing. We spent a lot of time and money working with a design firm to establish the current brand across all CNCF sites and it would be a shame to allow it to degrade:
|
|
Thank you, I didn't know these background details. But my concern is that many CNCF sites still have the old docsy theme remaining in their repositories as git submodules. Given compatibility with Node, Hugo, Netlify, etc., it wouldn't be surprising if these break at some point. Is there a plan to update these across the entire CNCF? If there are already guidelines or like this for migrating while preserving the style, I'd be happy to follow them |
|
I don't think there's a plan yet to upgrade these sites. |
|
Thanks, anyway, wouldn't it be a good idea to get ahead on updating docsy on this site? I'm welcome to contribute on this topic. Please lmk if there's a need to discuss this within CNCF (I can participate in the discussion if needed) |
|
Also, please wait for a bit for fixing some part. |
|
I'm not sure I see the need to keep the docsy theme up-to-date, especially since the new versions break the UI of the site. Are there particular features you're wanting from the new version of docsy? Or did it solve some security issue? I'm all for staying up on the latest versions of core software libraries but, since docsy is just a theme, I wonder whether the same principle applies. |
|
From a maintenance perspective, I definitely want to remove the docsy submodule from the repository and update to docsy 0.7. This is because migrating to Bootstrap 5 is critical for security (even sites served as SSG can be affected by XSS, etc.). c.f. As you mentioned, we would reconsider whether updating to 0.13.x (docsy latest) is necessary. Initially, I also intended to update to the 0.7.x (as indicated by the branch name), but search and several features didn't work. After testing several versions, updating to 0.13.x resolved the issues, so I proceeded with that update |
|
However, from an accessibility perspective, updating to the latest Docsy version from 0.7.x is well-justified. Looks CNCF Accessibility Statement commits to WCAG 2.1 Level AA. The Docsy 0.7 ->0.13 bump includes some improvements that directly support this commitment:
|
|
So I think we need to consider the trade-offs between these (target-based) accessibility improvements and the CNCF brand image and design. Do you have any thoughts on this point? (Of course, it would be ideal if these weren't trade-offs, but I think it's difficult to make updates without any changing the site's structure at all.) |
|
If there are any accessibility improvements we can make, could you start a new issue that lists them and how we might be falling short of the standard? Back when we came up with the brand for CNCF we made a lot of effort to be compliant but may have missed one or two points. In some cases we may have deliberately made a small concession on accessibility for the sake of the design. I agree we should update Bootstrap if that fixes a security issue. Is it necessary to update docsy to accomplish that or can we go directly and update Bootstrap ourselves? |
|
Thanks, I'll create a issue, so please wait for a bit 🙇 |

Describe your changes
layouts/_partials/for Hugo compatibility with Docsy 0.7+Related issue number or link (ex:
resolves #issue-number)close #3609
Checklist before opening this PR (put
xin the checkboxes)git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to yourgitconfigby runninggit config --global alias.ci "commit -s".