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

Improve flow of install page #274

Merged
merged 2 commits into from
Nov 23, 2023

Conversation

dstansby
Copy link
Contributor

Description

  • If I'm installing the bundled app, I don't care about pre-requisites for installing the Python package, so move that section under the "Installing as a Pytho package" section
  • Modify the header levels, so the second level now only has "Install as Python package", "Install as bundled app", "Next steps". In particular, two sections which are only relevant to the Python package route have been moved down a level to simplify the top level contents.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 20, 2023
@melissawm melissawm added this to the 0.5.0 milestone Nov 20, 2023
@melissawm
Copy link
Member

It looks like the pr-milestone-check is causing the CircleCI redirector action to be skipped. In any case, this looks good to me, thanks @dstansby !

@psobolewskiPhD psobolewskiPhD modified the milestone: 0.5.0 Nov 20, 2023
@psobolewskiPhD
Copy link
Member

The whole CircleCI job isn't proc'ing...

Copy link
Member

@psobolewskiPhD psobolewskiPhD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@melissawm
Copy link
Member

Love the tabs, maybe in a follow-up?

@psobolewskiPhD psobolewskiPhD modified the milestones: 0.5.0, 0.4.19 Nov 22, 2023
@psobolewskiPhD psobolewskiPhD merged commit d3b9b68 into napari:main Nov 23, 2023
6 checks passed
melissawm pushed a commit to melissawm/napari-docs that referenced this pull request Dec 1, 2023
# Description
Update installation guide:
- If I'm installing the bundled app, I don't care about pre-requisites
for installing the Python package, so move that section under the
"Installing as a Pytho package" section
- Modify the header levels, so the second level now only has "Install as
Python package", "Install as bundled app", "Next steps". In particular,
two sections which are only relevant to the Python package route have
been moved down a level to simplify the top level contents.

Co-authored-by: Peter Sobolewski <76622105+psobolewskiPhD@users.noreply.github.com>
jni pushed a commit that referenced this pull request Jan 9, 2024
# References and relevant issues

This came up at the docs workgroup and also:
#274 (review)

# Description

Currently, the install docs have pip, conda and GitHub install
instructions in series followed by the `checking that it worked`. No-one
will use all three approaches.
In this PR instead Tabs are used for *each* of the approaches, to let
the user choose an install strategy.
This makes the install page simpler and cleaner, leading the user to the
the next logical step.

What it looks like:
<img width="776" alt="image"
src="https://github.com/napari/docs/assets/76622105/3c86bb63-07e0-44e9-83c5-6e777698f2db">
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants