Avdan-OS / Docsite

For documentation
13 stars 11 forks source link

Dev ---> Main #68

Closed StoneMcYT closed 2 years ago

Xapu1337 commented 2 years ago

I disagree with the switch usage (imo having indentation is necessary) and the number of tabs: 2 looks better than 4, it's less bigger

I would understand it for HTML, but for code? You kinda sometimes want some space between everything (useful for something like builders)

Froxcey commented 2 years ago

I disagree with the switch usage (imo having indentation is necessary) and the number of tabs: 2 looks better than 4, it's less bigger

I would say 2 spaces indention is more of a HTML thing. If your a code has too much indentation that you want to use 2 spaces, the nesting is probably too heavy and should be cleaned up a bit.

Edit: typo

ghost commented 2 years ago

I disagree with the switch usage (imo having indentation is necessary) and the number of tabs: 2 looks better than 4, it's less bigger

I can change the documentation of the switch, but about the 4 indentation spaces you're absolutely wrong. In c, c++ and rust the indentation spaces by default are 4.

Golem642 commented 2 years ago

I disagree with the switch usage (imo having indentation is necessary) and the number of tabs: 2 looks better than 4, it's less bigger

I can change the documentation of the switch, but about the 4 indentation spaces you're absolutely wrong. In c, c++ and rust the indentation spaces by default are 4.

I mean that's just my opinion, i like it like that. But ok, if you guys want it to be for then let it be

Froxcey commented 2 years ago

It's called convention, Rust uses 4 spaces.

Golem642 commented 2 years ago

It's called convention, Rust uses 4 spaces.

Okok i got it just please at least indentation for switch

ghost commented 2 years ago

Okok i got it just please at least indentation for switch

Why is it important having public and private keyword indentation?

Sammy99jsp commented 2 years ago

Please don't use this PR for discussion about particular changes -- this PR is to only update the main site.