Skip to content

Publish updates from master - #15128

Merged
usha-mandya merged 21 commits into
publishedfrom
master
Jul 15, 2022
Merged

Publish updates from master #15128
usha-mandya merged 21 commits into
publishedfrom
master

Conversation

@aevesdocker

Copy link
Copy Markdown
Contributor

Proposed changes

Related issues (optional)

@netlify

netlifyBot commented Jul 15, 2022

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

NameLink
🔨 Latest commitb603428
🔍 Latest deploy loghttps://app.netlify.com/sites/docsdocker/deploys/62d1d924935aeb000967cd4a
😎 Deploy Previewhttps://deploy-preview-15128--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Add additional keywords to the page, which helps with the site's search
auto-complete, and fixes a typo (Dektop -> Desktop)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- fix some headings having the wrong level
- adjust headings to use the same casing as the UX
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Use a relative link to the markdown file to allow IDEs to verify the
link and anchor.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- wrap bullets
- align wording of common bullets to be the same between platforms
This makes it easier to see the differences between platforms when
comparing the files.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- Reformat the markdown syntax, and wrap long lines
- Slight rephrasing of common content to align between platforms,
or to make it more generic (Windows/Mac -> your machine).
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- Reformat the markdown syntax, and wrap long lines
- Slight rephrasing of common content to align between platforms,
or to make it more generic (Windows/Mac -> your machine).
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- sync the content in the software updates section between
platforms. as they only _slightly_ differed (Windows and
Mac are the same, Linux slightly different)
- remove "Disable automatic check for updates" note from
linux, as it was a duplicate of the paragraph below it.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This option has been available for over 6 months, so probably ok to
remove it as a note, and just keep the documentation how to use it.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
The "-settings" suffix in the URL duplicated the "/settings/" directory
in the URL, which made it redundant.
Using "/mac/", "/windows/" and "/linux/" keeps the URLs shorter, more
memorable, and more consistent with other locations, making it easier
to "guess" the URL when navigating using the browser's location bar.
No redirects were added, because the "-settings" pages have not yet
been published.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
thaJeztah
thaJeztah previously requested changes Jul 15, 2022

@thaJeztahthaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Left some comments; I was writing along when making them; let me push those suggestions as a pull request for review

(if we want to remove the -install suffix, it's best to make those changes first before publishing, otherwise we'll have to add redirects)

Comment thread_data/toc.yaml Outdated
Comment threaddesktop/settings/linux-settings.md Outdated
Comment threaddesktop/settings/linux-settings.md Outdated
Comment threaddesktop/settings/linux-settings.md Outdated
Comment threaddesktop/settings/linux-settings.md Outdated
Comment threaddesktop/settings/mac-settings.md Outdated
Comment threaddesktop/settings/mac-settings.md Outdated
Comment threaddesktop/settings/linux-settings.md Outdated
Comment threaddesktop/settings/mac-settings.md Outdated
Comment threaddesktop/settings/windows-settings.md Outdated
thaJeztahand others added 2 commits July 15, 2022 13:28
Looks like these were not intentional
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@usha-mandya
usha-mandya dismissed thaJeztah’s stale reviewJuly 15, 2022 21:23

Merged @thaJeztah's PR which addresses the suggestions and comments added on this PR.

@usha-mandya
usha-mandya merged commit b8a86dc into publishedJul 15, 2022
Sign up for freeto 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.

4 participants

@aevesdocker@thaJeztah@usha-mandya@craig-osterhout