Skip to content
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

Add /try to the nav #528

Merged
merged 2 commits into from
Jan 6, 2022
Merged

Add /try to the nav #528

merged 2 commits into from
Jan 6, 2022

Conversation

palewire
Copy link
Collaborator

It is a bit unusual the /try page isn't in the nav. It seems to me like perhaps the first place you might want a newbie to find.

Is there a reason it isn't there now? I propose we add it as part of the gradual cleanup sparked by the discussion in #433 and as a companion to #497.

@choldgraf
Copy link
Collaborator

I think this is a reasonable change, with two caveats:

  1. (minor) We've already got a lot of nav links in the top navbar, so I worry we're starting to clutter it up a bit
  2. (major) We have Temporarily use JupyterLite for high-traffic links to mybinder.org #513 still unresolved, and try is generating a ton of traffic to mybinder.org already that is generating a high cost. It would be really nice if we had a resolution to this before making try even more prominent

@palewire
Copy link
Collaborator Author

Makes sense. Another argument to merge #497 and combine the events and get involved pages per #499. You agree?

@choldgraf
Copy link
Collaborator

Yep that sounds reasonable to me!

@choldgraf
Copy link
Collaborator

Though for some of these, I think we should point people to the jupyter-wide docs at jupyter.readthedocs.io. those are a bit more in depth and more like proper documentation, while jupyter.org is more like a brochure site (at least I think that is how they relate to one another, they both predate me!)

@palewire
Copy link
Collaborator Author

palewire commented Dec 24, 2021

Alrighty. I'll get around to merging Events and Get Involved before too long. Along those same lines, there is an open pull request now to switch the Documenation nav to just such a link off. #507

@palewire palewire linked an issue Dec 27, 2021 that may be closed by this pull request
3 tasks
@palewire palewire changed the title Add Try to the nav Add /try to the nav Dec 27, 2021
@Carreau
Copy link
Member

Carreau commented Jan 6, 2022

Ok, let's try.

@Carreau Carreau merged commit f52a834 into master Jan 6, 2022
@Carreau
Copy link
Member

Carreau commented Jan 6, 2022

pun was not intended.

@choldgraf
Copy link
Collaborator

lol

@krassowski krassowski deleted the palewire-patch-17 branch May 26, 2024 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants