Diskusie » Spätná väzba ku Greasy Fork
Code Diff should ignore leading/trailing spaces
Just saw the checkbox! Maybe it should be enabled by default? Meanwhile I'll write a personal script that enables it for me...
The current HTML of the checkbox is <input id="ignore-whitespace" type="checkbox" name="w" value="1"> which looks like a mistake because checkboxes don't use value, they should have checked like this: <input id="ignore-whitespace" type="checkbox" name="w" checked> so I guess it's a bug, @JasonBarnabe?
The diff shown inside reports doesn't have this checkbox.
checkboxes don't use value
Sure they do, it's the value sent to the server when checked.
The diff shown inside reports doesn't have this checkbox.
I've changed it in reports so that it ignores whitespace by default.
Many scripts change the indentation or switch tabs/spaces so the built-in diff becomes useless and we have to compare in an external tool, which is very time-consuming when doing it more than once...