Closed hyounes4560 closed 2 years ago
Thanks @hyounes4560 for all your hard work on this! I apologize for my absence over the past week; my mom's living situation became unstable again and I had to go help her move (again).
Per your request, I went ahead and took care of the minor changes as commits on my side. The changes are:
:menuselection:
syntax) and applying reST formatting a few other placesI broke them up into separate commits for your review, but they'll all be squashed into your commit when this is merged. Thanks!
Looks good; well done, Hanan! No further comments from me.
Pro tip—I find it a good habit to always git pull
before I make any local changes to pick up any remote ones you or others pushed the the PR, as it avoids the merge conflict and related issues you ran into here (as otherwise, if I don't make it a habit, I find it easy to forget).
Also, quick reminder, you can apply multiple suggestions at time with the same message by (on the Files
tab) clicking Add to batch
on each suggestion and then Commit
once you'd added all of them.
So long as we're squash merging, neither of these issues poses major problems at our end—but it can certainly help things go smoother on yours :)
Pull Request
Pull Request Checklist
Description of Changes
Add a new section for the Outline pane as per @CAM-Gerlach suggestion.
Issue(s) Resolved
Fixes #326