Sleazy Fork is available in English.

討論 » Greasy Fork 回應

Code Diff should ignore leading/trailing spaces

wOxxOm管理員
§
發表於:2021-05-10

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...

wOxxOm管理員
§
發表於:2021-05-10

Just saw the checkbox! Maybe it should be enabled by default? Meanwhile I'll write a personal script that enables it for me...

wOxxOm管理員
§
發表於:2021-05-10

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?

wOxxOm管理員
§
發表於:2021-05-10

The diff shown inside reports doesn't have this checkbox.

JasonBarnabe管理員
§
發表於:2021-05-10
checkboxes don't use value

Sure they do, it's the value sent to the server when checked.

JasonBarnabe管理員
§
發表於:2021-05-11
The diff shown inside reports doesn't have this checkbox.

I've changed it in reports so that it ignores whitespace by default.

發表回覆

登入以回復