aarpa / prj-rev-bwfs-dasmoto

0 stars 0 forks source link

Keep lines short #2

Open sangjunp4rk opened 6 years ago

sangjunp4rk commented 6 years ago

Your lines are a bit long right now. Try splitting this into new lines so it's easier to read on one screen without scrolling. Convention is usually ~79 characters per line. It’s an issue of code density. Programmers like to have a lot of context when reading code. It is much, much quicker to glance up and down a window than it is to scroll. If you have very long lines, you also tend to have lines that vary in length a lot, leading to a lot of wasted screen real estate and being able to fit less code on screen at a given time overall. Don’t always rely on auto-wrapping since that’s not present in all text editors.

https://github.com/aarpa/prj-rev-bwfs-dasmoto/blob/master/Dasmoto/index.html#L32

aarpa commented 6 years ago

I used Atom as my text editor, and I used the Toggle Soft Wrap feature so that I was able to view the paragraph entries on one screen without scrolling. I'm assuming this meant that viewing my code on a different text editor would then remove that soft wrap? If so, how can I ensure that it stays wrapped when it's being reviewed by another developer?

https://github.com/aarpa/prj-rev-bwfs-dasmoto/blob/master/Dasmoto/index.html#L32