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: Update Home Page and Setup Guides #478

Merged
merged 17 commits into from
Jan 3, 2022
Merged

docs: Update Home Page and Setup Guides #478

merged 17 commits into from
Jan 3, 2022

Conversation

prcutler
Copy link
Contributor

Hi,

These are my first commits to Mainsail's docs, so if I didn't do something right or you don't like the changes, I won't be offended! :) Just let me know what you would like changed and I'm happy to help.

I've gone through the home page and the setup guide for edits. As I'm getting used to the tone and style of the docs, I just start by reading and updating what's there.

Generally the way I've been taught is think about translations and accessibility when writing docs. I try to limit contractions (don't vs do not), use simpler language to make it easier for translation, and limit parentheses ().

I'll keep reviewing and sending some edits in and then I'll start helping write some of the newer chapters for the next release.

Thanks for reviewing and let me know the feedback for what I should change.

```yaml
[display_status]
```

## Pause, Resume, Cancel
This enables pause / resume in mainsail.
These macros control the current print job.
Copy link
Member

Choose a reason for hiding this comment

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

[pause_resume]is only to enable the PAUSE/RESUME commands in klipper...

@prcutler
Copy link
Contributor Author

Hi,

I've reviewed and edited all of the Update guides. Thank you to whomever wrote the documentation, it's always easier to edit than create!

I've tried to improve the spelling and grammar to improve readability. I've also tried to remove contractions (don't vs do not), shorthand (config vs configuration), file names, etc. to make it easier for translators.

If there any mistakes or if anything doesn't make sense, please let me know and I'm happy to fix them. I can take the feedback. :)

@meteyou meteyou changed the title Update Home Page and Setup Guides docs: Update Home Page and Setup Guides Dec 31, 2021
docs/index.md Outdated
- Temperature Presets (with custom gcode)
- Bed Mesh Leveling
- Multi-Webcam support
- Control power devices (relays, TPLink, Tasmota, ..)
- Customizable UI (logos, backgrounds & even custom css)
- Control power devices such as relays, TPLink and Tasmota devices, and more)
Copy link
Contributor

Choose a reason for hiding this comment

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

remove ) at the end

docs/index.md Outdated
- Control power devices (relays, TPLink, Tasmota, ..)
- Customizable UI (logos, backgrounds & even custom css)
- Control power devices such as relays, TPLink and Tasmota devices, and more)
- Customizable user interface includinglogos, backgrounds, and custom CSS
Copy link
Contributor

Choose a reason for hiding this comment

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

includinglogos -> missing whitespace

```bash
cd ~/klipper
git pull
```

Restart Klipper (`sudo service klipper restart`) and check the `klippy.log`, if Klipper starts
correctly and then continue the guide.
Restart Klipper using the command `sudo service klipper restart` and check the `klippy.log`. if Klipper startsed correctly without any errors you may continue.
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: startsed

@prcutler
Copy link
Contributor Author

prcutler commented Jan 1, 2022

Thanks! Sorry I missed some of these. I'll get the changes in by tomorrow.

Copy link
Contributor

@lixxbox lixxbox left a comment

Choose a reason for hiding this comment

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

I made two small comments. if they are fixed, i think it can be merged.
Enabling the update manager via the mainsail editor is something we can do later.

@meteyou meteyou merged commit a339cab into mainsail-crew:develop Jan 3, 2022
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