-
Notifications
You must be signed in to change notification settings - Fork 165
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
Coverage website looks weird #1932
Comments
nodejs/nodejs.org#2544 looks like a likely suspect. |
I think #1933 should update the right place. |
https://benchmarking.nodejs.org/ also looks affected. I've raised nodejs/benchmarking#289. |
Sorry, I couldn't know that the styles were used from other sites :/ I think the best solution would be to redirect styles/css and eventually update those sites to use the new path to styles CSS. |
Alright, I made #1935 which should fix this. But, I could definitely use a list of sites that use the assets from the main site. Which BTW is bad practice for this reason, especially the CSS. I could work on just adding the needed CSS on each site and also make sure everything is working fine; I just need a list of sites. |
@XhmikosR in the past when we've done major reworking of the website, we've used redirects to keep things consistent. I wouldn't like to have a massive list of redirects but the overhead isn't terrible so the list could grow. See toward the bottom of the config file @ https://github.com/raw/nodejs/build/master/setup/www/resources/config/nodejs.org (in this repo, setup/www/resources/config/nodejs.org, not bothering to link because GH wants to display it as markdown). Perhaps this would be a good thing to do for any significant resources that have been moved? CSS, images I guess. benchmarking and coverage are the main sites that reuse the CSS I think. If I can be indelicate, I believe the reason for the linking I think is that the people that manage those sites aren't designers and are just happy to reuse consistent styles. I suspect the maintainers of both of those sites would be more than happy to receive some design attention for their sites, including, perhaps, restructuring of resources and where they are located. |
@rvagg I already have #1935 which adds the redirects. I think the easiest short-term solution, without risking to break things again, will be to include the CSS in those repos. This way it won't be affected by an upstream change. If there's an upstream change, changes will be needed anyway. I'll think about it and maybe provide PRs on each repo. For the time being #1935 should fix the breakage by adding redirects. |
ah, got it, sorry, missed #1935 in my notifications scan |
@XhmikosR I searched the org for I think https://coverage.nodejs.org/ used to be generated from nodejs/testing but that repository is now archived and the scripts moved to nodesjs/build. |
Ok, so it's just these 2 websites, so with the current open PRs things will be fixed. We can revisit how to handle the CSS case later. |
https://coverage.nodejs.org/ looks odd, the header image is huge and styles are mostly missing. I think this is happening because https://nodejs.org/layouts/css/styles.css 404s, but I can’t tell whether that’s a build or a website issue (there were some recent changes to the nodejs.org repo that likely affect this but I’m not sure where the fix should end up, here or there). /cc @XhmikosR
The text was updated successfully, but these errors were encountered: