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

Suggest user friendly edits for Extend chapter #459

Merged
merged 1 commit into from
Aug 16, 2015

Conversation

willingc
Copy link
Contributor

Hi! I was reviewing the tutorial for mentoring DjangoGirls at DjangoCon 🎉.

I stumbled a bit over the Extend Your Application chapter because I kept losing my place in the page scroll. There's a lot of code snippets that look very similar and very few section headings as a visual cue for where you in the page.

I suggest adding a few subsection titles and a few other edits to be a bit more specific with terms like "it". Please use what you think might be helpful. I like this section quite a bit and making it more user friendly would help 🌻 🐰 🌷

Hi! I was reviewing the tutorial for mentoring DjangoGirls at DjangoCon 🎉.

I stumbled a bit over the Extend Your Application chapter  because I kept losing my place in the page scroll. There's a lot of code snippets that look very similar and very few section headings as a visual cue for where you in the page.

I suggest adding a few subsection titles and a few other edits to be a bit more specific with terms like "it". Please use what you think might be helpful. I like this section quite a bit and making it more user friendly would help :D
asendecka added a commit that referenced this pull request Aug 16, 2015
Suggest user friendly edits for Extend chapter
@asendecka asendecka merged commit f2bf7b0 into DjangoGirls:master Aug 16, 2015
@asendecka
Copy link
Contributor

Amazing work @willingc! You rock :D. ⚡ ⭐ 🍓 🍒 🏁

@willingc willingc deleted the patch-1 branch August 16, 2015 11:43
@willingc
Copy link
Contributor Author

Thanks for the merge @asendecka 🌊

Closes #460

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.

2 participants