-
Notifications
You must be signed in to change notification settings - Fork 4
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
Clicking in the Blosc logo should redirect to the home page #4
Comments
I agree. This doesn't need to be implemented here bit in the other repos. Besides the logo going to the global home, I think the navigation bar should have the same menu options and the same look. I'll have a look. |
I see that the top navigation bar in sub-projects (C-Blosc2, Python-Blosc, Python-Blosc2) is not showing up anymore. Also clicking in the Blosc logo redirects you to a non existing place. I think this is still work in progress, but FYI. |
Yes, I realized on Friday, I'll fix it today. I couldn't test the logo link well until everything was put together. It's a bug in the sphinx theme, but I can implement a workaround. For the navigation, I'm checking what's best. Sphinx is limiting the options, but I think I can do something that should make navigation nice and intuitive. Will ping you a bit later with the PRs. |
I opened 3 PRS, one for each project, fixing the logo link, and the navigation (which will show in the sidebar, the top bar will have the same links as the website/blog): This is what makes more sense to me, but after you test it, let me know if anything doesn't make sense to you. |
Thanks. @martaiborra could you do a review on this, and if if looks good to you, please merge the different PRs. |
That is taking a really good shape. One minor thing: when clicking on the github button of the navigation bar, one is always redirected to https://github.com/Blosc/c-blosc2, irregardingly of the project that one is browsing. If it is not possible to send to the different projects, maybe it is better to redirect to https://github.com/Blosc. Thanks. |
My bad. They can be different, I just copy-pasted the settings and forgot to update it. I just opened PRs to fix them. |
Ok. After that, I think the only pending task is to update links to the new documentation site and remove the readthedocs projects. Correct? |
I already updated updated the links to the docs in the READMEs, those are the only ones I'm aware of. If there are more links to the old docs, yes, that'd be pending. Copying an updated version of this comment with the things that I'm aware of:
The secret |
Thanks for the reminder. I am not sure we want to activate actions for trigger the docs rendering in the 3 projects. Right now, I'd prefer to deploy the docs manually during the releasing process (see e.g.: Blosc/python-blosc2@7e2403f#diff-e71b27ab83b907d230fca9db0a00150a9c686a2bb8465646d5b5a007de7946fbR55-L58). This introduces less 'magic' and IMO is less error prone in the long run. Will take care of the other 2 points though. |
Sounds good. If useful, we can add workflow_dispatch to the CI here, so you can build the docs via the GitHub UI or API. One thing we discussed but still need to be addressed... This is now set up to build the docs of the main/master branch of each repo. We can easily add to the CI a checkout of the latest tag (in chronological order) with |
Ok. So I tried to use redirections using the instructions at: https://docs.readthedocs.io/en/stable/user-defined-redirects.html. After a lot of attempts, I was really unable to make them working properly. After having a look at readthedocs/readthedocs.org#8143 (the NumPy folks also tried to do the same recently), they also found quite a lot of issues (reading the ticket is quite instructive actually), I decided to entirely remove the projects (C-Blosc2, Python-Blosc and Python-Blosc2) from readthedocs. I just hope I did not fucked too many things in doing this. |
BTW, I noticed that the URLs for the new projects are like: https://www.blosc.org/python-blosc2/python-blosc2.html. Would it possible to reduce them to just https://www.blosc.org/python-blosc2 (and the same for others)? That would be much shorter for docs and the like. |
The problem comes from how the Sphinx template builds the header. What it does is check the table of contents in the root page, and adds the items there. Then you can append any other link you want to the navigation. What I did is to add just an element to the root page, that points to a second page with the README and the table of contents to the actual sections. A quick fix for the links it to rename the Sphinx root to something different than the default index.html, and make the page with the README be index.html, so that page would be just Something that can make more sense is to leave Sphinx with the default behavior, but try to change the template it uses, so we add the website navigation on top of the Sphinx navigation. There would be two navigation bars then, one on top of the other, something like:
I think this would be a much cleaner solution. I don't think it's so complicated, but it can probably take couple of days of work. And I don't have the time right now for it. For fixing only the links for now, it should be as easy as changing in the Sphinx
Then rename |
For the redirects, there is an option I wouldn't personally implement, that is using JavaScript redirects. That would mean embedding some html in each Sphinx page with a javascript redirect to the new page. For users arriving to the website, the old page would render, and once fully loaded the page would redirect to the new location. The problem of doing it with JavaScript and not with the proper HTTP headers is that search engines would not update the index, as they are not aware of JavaScript redirects (afaik). It's unfortunate that readthedocs redirects don't work (I never used readthedocs myself), but I think what you did is the best. For some days the links in search engines and users benchmarks will be broken, but after these few annoying days everything will be in place, with pages properly indexed, probably better than before, as all the Blosc pages now live under the same domain, and search engines should give it more important for the relevant keywords. |
I have been trying to look how to do the redirection for python-blosc.blosc.org in our nginx, but this is taking more time than I'd like to use here. For the time being, I have just disabled the python-blosc.blosc.org entirely. I think @datapythonista is right and that after some initial days where people is looking for docs, they will google for the new site. Damn, this latest redirection part went a bit crazy, but hey, the result is very good already. |
If python-blosc is hosted in your own nginx, I think the redirect should work by simply adding this to the
|
I gave a try to changing Sphinx |
Right now, if we are in say, https://www.blosc.org/python-blosc2/release_notes/index.html, and click to the blosc logo in upper left, this redirects you to: https://www.blosc.org/python-blosc2/index.html. I think redirecting to https://www.blosc.org is much better in terms of navegation. And the same applies to the C-Blosc2 and Python-Blosc doc pages.
@datapythonista what do you think?
The text was updated successfully, but these errors were encountered: