search

cksource-archive / samples-framework

☠☠☠ This repository is no longer used ☠☠☠
Other
0 stars 0 forks source link

Fix all spaces in code snippets #38

Closed adelura closed 9 years ago

adelura commented 9 years ago

Should be tabs instead of spaces.

Reinmar commented 9 years ago

https://github.com/cksource/samples-framework/commit/5697a05ddc8f647c54d45b2efe69b3e964880704#diff-634f98626494af09e9783e1f5e0c045dR155

2? We do use 4.

Reinmar commented 9 years ago

Besides that there's no point in defining -o-tab-spaces.

adelura commented 9 years ago

Opera mini and opera mobile requires -o frefix.

Reinmar commented 9 years ago

I don't think that Blink based Operas require now the -o prefix.

adelura commented 9 years ago

http://caniuse.com/#search=tab-size click "Show all" button.

Reinmar commented 9 years ago

Uh, that's sad. Ok, you're right then.

Reinmar commented 9 years ago

I mean - it's only because of Opera Mini, but it doesn't really mean anything for us, since it's not supported by us.