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 prepare themes page with review feedback #554

Merged
merged 5 commits into from
Jan 20, 2022

Conversation

prcutler
Copy link
Contributor

Fix the paragraph on preparing the custom theme with lixxbox's feedback (thanks!)

@lixxbox lixxbox self-assigned this Jan 20, 2022
@lixxbox lixxbox added the Is: Docs Anything referencing documentation label Jan 20, 2022
You can use Mainsail's built-in file manager `(Settings > Machine)` to create the .theme folder and upload all the files and edit your css-files.

The dot in the folder name means that it is a hidden folder. Therefore you have to activate "show hidden files".
The dot in the folder name means that it is a hidden folder. You will need to activate "Show Hidden Files" in your operating system.
Copy link
Contributor

Choose a reason for hiding this comment

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

I am still not sure what is meant here by operating system. The setting is made in Mainsail's file manager.


All listed files in the .theme folder are optional and will only be loaded if they are provided.
## Directory structure
The below directory and file strucutre provides an overview of files you could include. All listed files in the `.theme` folder are optional and will only be loaded if they are provided.
Copy link
Contributor

Choose a reason for hiding this comment

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

strucutre -> structure

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.

Please explain (or change) the Operating System in this context and fix the small typo.

@lixxbox lixxbox merged commit 69bb61a into mainsail-crew:develop Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Is: Docs Anything referencing documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants