Timeline for How to handle user rollbacks due to stubbornness about formatting / content?
Current License: CC BY-SA 3.0
25 events
when toggle format | what | by | license | comment | |
---|---|---|---|---|---|
Jun 2, 2014 at 6:39 | comment | added | Cerbrus | @SargeBorsch: the reason I added those whitespaces, was to make it easier to spot the differences between those 4 lines, which was basically what the question was about. In real projects, sure, it's evil, but on SO, it makes the code that much easier to understand, sometimes. | |
Jun 1, 2014 at 11:32 | comment | added | Barney | Aggressive automatic whitespace reformatting in IDEs and version control tools & setups that can't distinguish between whitespace changes and code changes are nasty historic problems. This isn't news or controversial. But editing whitespace on a StackOverflow post for readability is super helpful. It's also a separate issue: StackOverflow integration into the IDE or code versioning is probably not an issue for the user, so we might even consider these entirely distinct problems. | |
May 30, 2014 at 23:38 | comment | added | cwallenpoole | @kurast Not convinced. The biggest issue is lining up text after a line break. It's downright nasty without monospace. | |
May 30, 2014 at 19:13 | comment | added | kurast | @cwallenpoole Because they look nice! See for yourself: nickgravgaard.com/elastictabstops/news/programming-fonts | |
May 30, 2014 at 17:16 | comment | added | cwallenpoole | @kurast AAAAHHHHG!! How... but... WHYI!?!?! | |
May 30, 2014 at 16:54 | comment | added | Ben Voigt | @SargeBorsch: "Makes code unmaintainable" Who cares? Code excerpts on StackOverflow are all about readability. Answers aren't supposed to be copy+paste ready to drop into someone's source file with no editing needed. Real code files need to abide by style guidelines. Questions and answers need to be readable. | |
May 30, 2014 at 11:00 | comment | added | Display Name | >"Oh here are some changes, looks really different, lets see.... nothing is different... just a bunch of WHITE SPACE AAAAARGGHHHHHHH" — yeah, this is annoying, especially if it causes a merge conflict. Grrrrrr! | |
May 29, 2014 at 21:15 | comment | added | AaronLS | @SargeBorsch I personally do some code aligning myself and there is som research showing how much quicker it is to read, but I certainly do not go behind old code or others and do this. Someone reviewing a checkin will be like, "Oh here are some changes, looks really different, lets see.... nothing is different... just a bunch of WHITE SPACE AAAAARGGHHHHHHH" and a keyboard dies. Additionally, many IDEs auto formatting will blow away that whitespace anytime someone touches that piece of code, making the effort wasted, or someone has to fight the IDE to maintain the whitespace. | |
May 29, 2014 at 19:56 | comment | added | Matt K | I agree that adding unnecessary whitespace is a bit silly in this context, but it is also silly for the OP to roll-back this seemingly harmless edit. You're both at fault of nitpicking, although I want to side more with you after reading the user's website claiming he's "the world's first Web 3.0 Professional Web Designer and OOP PHP Web Developer"... sigh... | |
May 29, 2014 at 19:36 | comment | added | Display Name | @kurast Nothing wrong in such practice. In fact, you're awesome. | |
May 29, 2014 at 19:29 | comment | added | kurast | @SargeBorsch To add some spice to the discussion, I would like to add that I program using a variable-width font (Oh the humanity!!). Garamond, to be exact. Vertical alignment does not exist for me (and is evil), except for the start-of-line tabs. When I post here, and use the code block, my code is always funny, because I use com.mycompany.thislibrary.LongAndDescriptiveObjectAndMethodNames that look nice and readable on my font, and become trash on the monospace font. However, I mostly never have to break a line because it easily handles 250 chars in a line. | |
May 29, 2014 at 18:28 | comment | added | John Nicholas | @PeterMortensen sure 20 upvotes and counting. 1) thats a minor edit 2) how is that really improving anything in anyway other than purely subjective 3) If you start editing like that in source control people are going to hate you. | |
May 29, 2014 at 14:18 | comment | added | Travis | @PeterMortensen I don't see that a source is necessary. There's a clear disagreement on what is correct, so one way should not be edited to the other. | |
May 29, 2014 at 13:33 | comment | added | Display Name | @PeterMortensen This makes code unmaintainable. | |
May 29, 2014 at 12:44 | answer | added | user164226 | timeline score: 13 | |
May 29, 2014 at 12:34 | comment | added | Joe W | What is the point of adding white space to lines to strings to get them to line up all pretty? | |
May 29, 2014 at 12:02 | comment | added | Peter Mortensen | @Sarge Borsch: do you have a source to back up that statement? | |
May 29, 2014 at 11:41 | comment | added | Display Name | "aligning" code (putting excessive whitespace in a middle of the string) is evil; please, don't do this never again | |
May 29, 2014 at 2:41 | answer | added | JumpingJezza | timeline score: 5 | |
May 29, 2014 at 1:13 | answer | added | John | timeline score: 7 | |
May 28, 2014 at 21:08 | vote | accept | Cerbrus | ||
May 28, 2014 at 19:53 | answer | added | Bruno | timeline score: 69 | |
May 28, 2014 at 19:17 | answer | added | nikib3ro | timeline score: 10 | |
May 28, 2014 at 14:22 | answer | added | Sam I am says Reinstate Monica | timeline score: 48 | |
May 28, 2014 at 13:31 | history | asked | Cerbrus | CC BY-SA 3.0 |