-
Notifications
You must be signed in to change notification settings - Fork 30
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
footer updates not showing up #524
Comments
@shawna-slh I know how to fix. Will open a PR in the next 5 min. |
@shawna-slh @iadawn Here is the PR to fix this: w3c/wai-website#383
|
@remibetin (cc @iadawn) It would be easier if the footer info lived in the repo with the document. What about removing the ‘footer’ entirely from the config.yml in the wai-website repo? |
@shawna-slh We would probably have to declare (and update) footer content in each page frontmatter if we want this info to live in this repo. I am not sure it is better. cc @iadawn |
That is problematic since all subpages may not be part of the same resource. Additionally, this approach makes it harder to check and less 'findable'. My strong preference is for this to be declared in the _config.yml of the resource as a default. This can be overridden in specific parts of the resource by including altered content in the header YML. |
Maybe the phrase "subpages" is misleading. To be more specific, a "Jekyll collection" has been used for this resource. All pages in "_policies" folder share common default attributes declared in
Unfortunately, to the best of my knowledge, the I think of two main options: (feel free to add a new one!)
|
Well that is a pain in the neck. My preference would still be to have this in the repositories rather than the main wai-website. Not idea but keeps it local. |
pull request
new footer in preview
missing from published resource footer
@iadawn before we announce this, I'd like to get this fixed. Maybe we ask for Remi (or Howard) to look into it?
The text was updated successfully, but these errors were encountered: