-
Notifications
You must be signed in to change notification settings - Fork 27
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
Update of the Solo Guide #179
Conversation
Fix discord link in footer
Title changes, shortened a lot of text, merged the acquire-grc-top and acquire-grc-bot classes in the .css file to the acquire-grc class.
Completely redid the Step 4 Section.
Shortened text troughout, made title capitalizaiton consistent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found a few things that need tweaking, other than that looking good so far.
Ment to do this in the commit "Remove The BAM Icon"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good PR, some minor typos need to be addressed. Also, please decide whether to use single or double quotes and stick with that choice throughout.
Converted all tabs to four spaces
Changes "a browser should come up" to "a browser should open"
Broke one sentence into two and changed the second sentence up
Reduced a lot of unneeded and repetitive text. Added in a note that fixes #113. Simplified some css, consolidating two classes into one. Helps somewhat with #27. In the future, I plan to overhaul the guides but this will take a while so I'm putting this update up for now.