2

In this edit suggestion somebody fixed the code formatting by indenting every line of code with one tab. In the markdown diff of the edit the tab characters are not shown, making it unclear what exact change was suggested there.

The markdown diff should show the tabs with some visible character, like it is done with spaces.

1

Fixed in latest, I now substitute tabs with 4 spaces ( ) for diff purposes.

  • 1
    This might not work when an editor changes erroneous tabs into spaces: it seems the reviewers won't see that change then? (The original had 4 SPACES <p... followed by TAB <a... which I think was changed into 4 SPACES <a.... I guess there is no way for the reviewer to see that? The rejection might still be fine, as 4 spaces make no difference. Given the original I guess either 2 tabs or 8 spaces should have been used.) – Arjan Apr 13 '11 at 10:39

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .