dwyl / book

📗 Our Book on Full-Stack Web Application Development covering User Experience (UX) Design, Mobile/Offline/Security First, Progressive Enhancement, Continuous Integration/Deployment, Testing (UX/TDD/BDD), Performance-Driven-Development and much more!
https://dwyl.github.io/book/
820 stars 134 forks source link

Transfer repo to dwyl simplify readme intro and focus on WIIFM #66

Closed nelsonic closed 7 years ago

nelsonic commented 7 years ago

This is only the beginning. We know exactly what we are doing in 2017!

@iteles please review/correct and when happy, merge. thanks! ❤️

iteles commented 7 years ago

🎉 🙌 Just finished reading, excited for the fresh start!

I pushed a commit:

I haven't touched the content itself.

Other than the comments I have added on specific lines, my only suggestion would be that we can evolve it to be even more concise and to-the-point. If you can give me a bullet point list of what you would like to get across, I can review this with an 'editor's eye' 👀 and make some effective cuts to the wording whilst making the key points stand out for people skim-reading. 👍 (in a separate PR)

I'm happy to merge as-is, but please give the changes in my commit a quick look-over before I do!

nelsonic commented 7 years ago

The reason for using the <br /> instead of the double space markdown is the sheer number of PRs we've had in the past two years in which people "remove trailing white space"... it needs to be explicit else people will remove the syntactic content breaks.

iteles commented 7 years ago

@nelsonic Agreed, I get that, I'm currently opening the issues from this PR and adding <br/>s consistently is one of them.

nelsonic commented 7 years ago

@iteles sweet! 🍭 😍