Refined-github: Remove whitespace toggle

Created on 2 May 2018  路  6Comments  路  Source: sindresorhus/refined-github

image

GitHub itself offers this functionality now.

enhancement help wanted

Most helpful comment

I would prefer to hide the native GitHub toggle.

All 6 comments

IMO thats a bad move from GitHub. I'm glad GH added that option but to perform some temporary action like "Toggling whitespace changes" takes 3 clicks instead of RGH's 1 click option.

Users are more likely to use the "Toggle whitespace" option in case of large PRs (where we can ignore whitespaces changes if linting tests pass), and less likely use "Unified/Split view" as it is persistent anyway.

But still, I'd like to know other people opinion too!

@notlmn Maybe hide the option by GitHub then if the extension is installed.

I would prefer to hide the native GitHub toggle.

I would prefer to hide the native GitHub toggle.

In that case, the current add-diff-view-without-whitespace-option needs to be changed to something like "set diff settings faster" and also extract the Unified/Split form

What if we just replace the text with one of these icons?

screenshot 2018-12-09 at 18 39 13

@bfred-it I like the icon idea, would we also want to hide the "Just for now" section, if the add-diff-view-without-whitespace-option is enabled?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

hkdobrev picture hkdobrev  路  3Comments

hkdobrev picture hkdobrev  路  3Comments

olso picture olso  路  3Comments

mareksuscak picture mareksuscak  路  3Comments

durka picture durka  路  3Comments