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

Move links out of navbar and into sidebar / other areas. #617

Closed
wants to merge 12 commits into from

Conversation

TimMonko
Copy link
Contributor

@TimMonko TimMonko commented Mar 9, 2025

Description

Follow-up / depends on #610. Closes #615

Description

I think the navbar is a bit too cluttered due to the 'More' section being propagated to the sidebar in smaller width screens (and generally being finnicky)

  1. Moves 3 lower-value (I still love them) links out of the navbar: blog, zulip chat, and workshop template
  2. Moves blog to homepage sidebar
  3. Moves workshop template link to Workshops page in Usage
  4. Removes zulip link in favor of presence in homepage sidebar and presence in footer. If we want to keep this in header we should move the footer link up to the header in theme

TO-DO:

  1. Add a blog link to Community
  2. Add troubleshooting / FAQ link (need to check if we have this yet lol)
  3. Add workshop link to Contributing?

TimMonko and others added 11 commits March 2, 2025 14:32
# Description
Add a -c option to suppress the detached head warning when building
docs, specifically when checking out a tag.
# References and relevant issues
Partially addresses napari#600 by improving the Advanced Topics landing page.

# Description
This PR adds cards to guide the reader to the npe2 specification. It
also more clearly indicates that napari plugin engine v1 is deprecated.
---------

Co-authored-by: Peter Sobolewski <[email protected]>
# Description
This PR updates the Circle CI orb and image for Python.

If this is accepted here, I will open a PR to update the `napari/napari`
repo as well.

Co-authored-by: Melissa Weber Mendonça <[email protected]>
Co-authored-by: Peter Sobolewski <[email protected]>
…i#609)

# References and relevant issues
<!-- What relevant resources were used in the creation of this PR?
If this PR addresses an existing issue on the repo,
please link to that issue here as "Closes #(issue-number)".
If this PR adds docs for a napari PR please add a "Depends on <napari PR
link>" -->
Closes napari#595

# Description
1. Changes plugin landing page tabs to link immediatelly to
'finding_and_installing_plugins' and the napari hub, rather than to the
index of start_using_plugins, which contained nothing more than the same
information now on the landing page.
2. Removes index from start_using_plugins because it is not needed
3. Updates TOC to remove the index and using finding and installing
plugins as top level header

<!-- Final Checklist
- If images included: I have added [alt
text](https://webaim.org/techniques/alttext/)
If workflow, documentation build or deployment change:
- My PR is the minimum possible work for the desired functionality
- I have commented my code, to let others know what it does
-->
# Description

This PR does the following:
- adds section headings to the contributing workflow for contributor
convenience
- reorders the contribution guides sections to reflect where new
contributors will commonly begin
- combines UI related contributions into one UI section with subsections

Co-authored-by: Peter Sobolewski <[email protected]>
@github-actions github-actions bot added documentation Improvements or additions to documentation maintenance CI, dependencies, and other maintenance labels Mar 9, 2025
@TimMonko
Copy link
Contributor Author

TimMonko commented Mar 9, 2025

I have no idea why so many files were edited? I know I rebased or something ... maybe that was the wrong thing to do? I thought that erased the commit history, not kept it, and it was merging that would have kept all the commits?

@TimMonko TimMonko marked this pull request as draft March 9, 2025 20:29
@psobolewskiPhD
Copy link
Member

Looking at the commit history it looks like you rebased main onto this branch rather than the other way? Hard to say without knowing your commands, maybe check your shell history and post what you entered?

@TimMonko TimMonko changed the title rename sidebar links file; add chat and release notes Move links out of navbar and into sidebar / other areas. Mar 10, 2025
@willingc
Copy link
Contributor

I think that you can git reset 9b16a58 to commit before the merge.

@TimMonko TimMonko closed this Mar 12, 2025
jni pushed a commit that referenced this pull request Mar 14, 2025
# References and relevant issues

Closes #615. Follow up of #610. Remake of #617 because I got confused
with the commit history and merging

# Description

I think the navbar is a bit too cluttered due to the 'More' section
being propagated to the sidebar in smaller width screens (and generally
being finnicky)

1. Moves 3 lower-value (I still love them) links out of the navbar:
blog, zulip chat, and workshop template
2. Moves blog to homepage sidebar (confirms there is a link to blog in
community)
3. Moves workshop template link to Workshops page in Usage, also adds a
link to Contributing index
4. Removes zulip link in favor of presence in homepage sidebar and
presence in footer. If we want to keep this in header we should move the
footer link up to the header in theme
5. Adds troubleshooting link to homepage sidebar (**check** that this is
internally referenced correctly, I can't do
https://napari.org/troubleshooting.html/ like I can for other internal
refs, it 404s)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation maintenance CI, dependencies, and other maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider whether to add more to the homepage sidebar
3 participants