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

docs: mkdocs index and nav redesign #5207

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

cwhite911
Copy link
Contributor

@cwhite911 cwhite911 commented Feb 25, 2025

[WIP] Redesign of docs index page and nav to improve overall site UX.

Related to Issue #5162

@github-actions github-actions bot added Python Related code is in Python CSS Related code is in CSS docs markdown Related to markdown, markdown files labels Feb 25, 2025
@cwhite911 cwhite911 self-assigned this Feb 25, 2025
@@ -26,6 +26,8 @@ theme:
- search.suggest
- search.highlight
- content.code.copy
- content.tabs.link
Copy link
Member

Choose a reason for hiding this comment

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

Can you please separate the tabs into a new PR to get it merged quickly, including the linked tabs option here? I was able to enable the tabs (locally), but the doc (mis)lead me to include some custom JS to keep the tabs in sync and that was not working for me.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Sorry it took me a minute to get back to this.

Copy link
Contributor Author

@cwhite911 cwhite911 Mar 4, 2025

Choose a reason for hiding this comment

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

Here is the PR: #5277

Copy link
Member

@wenzeslaus wenzeslaus left a comment

Choose a reason for hiding this comment

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

I like the approach!

Comment on lines 30 to 46

Starting GRASS GIS...

__________ ___ __________ _______________
/ ____/ __ \/ | / ___/ ___/ / ____/ _/ ___/
/ / __/ /_/ / /| | \__ \\_ \ / / __ / / \__ \
/ /_/ / _, _/ ___ |___/ /__/ / / /_/ // / ___/ /
\____/_/ |_/_/ |_/____/____/ \____/___//____/

Welcome to GRASS GIS 8.5.0dev (5ffc9ea669)
GRASS GIS homepage: https://grass.osgeo.org
This version running through: Bash Shell (/bin/bash)
Help is available with the command: g.manual -i
See the licence terms with: g.version -c
See citation options with: g.version -x
If required, restart the GUI with: g.gui wxpython
When ready to quit enter: exit
Copy link
Member

Choose a reason for hiding this comment

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

Maybe little too many lines? (That output should be shorter, but even a shorter one would still be little too long for here.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've change the section.


```bash

grass -c EPSG:3385 {project directory} --gtext
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
grass -c EPSG:3385 {project directory} --gtext
grass -c EPSG:3385 {project directory} --text

For true no-GUI experience.

Comment on lines +60 to +63
### Python Scripts

The `grass.script` module provides a Python interface to GRASS. This allows
users to write Python scripts to interact with GRASS. The `grass.script` module
Copy link
Member

Choose a reason for hiding this comment

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

We need to start describing the in-session version grass.script.setup.init.

Comment on lines 53 to 58
Now you can run GRASS commands in the terminal.

```sh
g.region raster=elevation
r.slope.aspect elevation=elevation slope=slope aspect=aspect
```
Copy link
Member

Choose a reason for hiding this comment

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

There is an alternative to this and that's --exec which I would say is popular for certain use cases.

Comment on lines 104 to 110
## Processing Tools

### GRASS Tool Prefixes

| Prefix | Category | Description | Link |
|--------|----------------------------------|------------------------------------|-------------------------------------------|
| `g.` | General | General GIS management tools | [General Tools](general.md) |
Copy link
Member

Choose a reason for hiding this comment

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

Just make this presentation of tool, rather than prefixes.

r.slope.aspect elevation=elevation slope=slope aspect=aspect
```

### Python Scripts
Copy link
Member

Choose a reason for hiding this comment

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

C doc and possibly general tool writing stuff (standard parser options): I wonder if to put it here as sort of next steps ("is this not enough?") or at the end to a separate section. This would be the usage as a modeling platform.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS Related code is in CSS docs markdown Related to markdown, markdown files Python Related code is in Python
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants