editorconfig (#1011) #9

Merged
zelvuska merged 4 commits from editorconfig into master 2 years ago
zelvuska commented 2 years ago
Owner
There is no content yet.
zelvuska added 2 commits 2 years ago
Poster
Owner

Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně.

Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně.
zelvuska reviewed 2 years ago
[*.py]
indent_style = tab
# I do not think we prescribe, how big a tab is.
indent_size = unset
Poster
Owner

A nechceme to nastavit? Protože pak člověk zarovná třeba komentáře, nebo kód k závorce, a jinde to bude rozjeté.

A nechceme to nastavit? Protože pak člověk zarovná třeba komentáře, nebo kód k závorce, a jinde to bude rozjeté.
Poster
Owner

A máme takový kód? Kromě římských čísel, která stejně žádným obecným pravidlem postihnout IMHO nejdou. A stejně obecně pokud potřebuju, aby to skutečně bylo zarovnané, tak musím použít stejné množství tabů na začátku a zbytek doplnit mezerami, jinak to „sémanticky“ je rozbité pořád (a nevidím úplně výhodu v tom snažit se před tím zavírat oči)…

Osobně jsem u Pythonu spíš názoru, že má existovat konečně mnoho úrovní odsazení a nesnažit se zarovnávat moc freestylově.

A máme takový kód? Kromě římských čísel, která stejně žádným obecným pravidlem postihnout IMHO nejdou. A stejně obecně pokud potřebuju, aby to skutečně bylo zarovnané, tak musím použít stejné množství tabů na začátku a zbytek doplnit mezerami, jinak to „sémanticky“ je rozbité pořád (a nevidím úplně výhodu v tom snažit se před tím zavírat oči)… Osobně jsem u Pythonu spíš názoru, že má existovat konečně mnoho úrovní odsazení a nesnažit se zarovnávat moc freestylově.
Poster
Owner

Á, to mi nedošlo, že by se to mělo zarovnat taby na odsazení a pak už jen mezerami. To máme asi někde špatně…

Á, to mi nedošlo, že by se to mělo zarovnat taby na odsazení a pak už jen mezerami. To máme asi někde špatně…
Poster
Owner

No, i tak by mě zajímalo, kde něco tak šíleného tvoříme. Na klasické věci mi zatím vždycky stačily klasické taby bez potřeby zarovnávat k závorkám; jediná věc, která mě napadá, jsou složité podmínky a tam mi (od stolu) dává smysl se spíš pokusit je zpřehlednit než udržovat…

No, i tak by mě zajímalo, kde něco tak šíleného tvoříme. Na klasické věci mi zatím vždycky stačily klasické taby bez potřeby zarovnávat k závorkám; jediná věc, která mě napadá, jsou složité podmínky a tam mi (od stolu) dává smysl se spíš pokusit je zpřehlednit než udržovat…
ledoian marked this conversation as resolved
Owner

Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně.

A máme někde vyloženě problém s odsazováním? To je aktuálně asi to jediné, co editorconfig umí sdělit…

> Napadá mě, jestli si nedefinovat i něco na html, protože některá vypadají strašně. A máme někde vyloženě problém s odsazováním? To je aktuálně asi to jediné, co editorconfig umí sdělit…
Poster
Owner

No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí).

No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí).
Owner

No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí).

Jakože v rámci jednoho souboru? Máš příklad?

Ano, výsledné HTML stejně bude maglajs, protože to DjangoTemplates hází do sebe bez ladu a skladu, ale to taky není problém editorů…

> No právě mi přijde, že jeden z problémů našich html je to, že jsou odsazované úplně náhodně a tak není vidět, kde která část končí. A trochu mi přijde, že tomu pomůže, když bude odsazování stejně široké (i když je to spíše problém toho, že jsou různé věci odsazené úplně náhodně vůči okolí). Jakože v rámci jednoho souboru? Máš příklad? Ano, výsledné HTML stejně bude maglajs, protože to DjangoTemplates hází do sebe bez ladu a skladu, ale to taky není problém editorů…
Owner

Ale jako whatever, pokud si myslíme, že tam mají být taby, tak to do toho configu napíšu :-)

Ale jako whatever, pokud si myslíme, že tam mají být taby, tak to do toho configu napíšu :-)
Poster
Owner

Tak třeba base.html je dost náhodné… Ale jak na to tak koukám, tak to není tak špatné, jak jsem měl pocit…

Tak třeba base.html je dost náhodné… Ale jak na to tak koukám, tak to není tak špatné, jak jsem měl pocit…
Owner

A co s těmi html chceme dělat? Taby šířky 2? 2 mezery? Nic širšího než 4 mi nedává smysl, protože to často je hodně zanořených bloků, i proto bych asi bral tu šířku 2, a tam už mi ty taby a mezery poněkud splývají…

A co s těmi html chceme dělat? Taby šířky 2? 2 mezery? Nic širšího než 4 mi nedává smysl, protože to často je hodně zanořených bloků, i proto bych asi bral tu šířku 2, a tam už mi ty taby a mezery poněkud splývají…
Poster
Owner

Dal bych 2 mezery (to je asi nejčastější, co tam aktuálně máme) (taby tam myslím nikde nejsou, oni se možná nelíbí htmlku...)

Dal bych 2 mezery (to je asi nejčastější, co tam aktuálně máme) (taby tam myslím nikde nejsou, oni se možná nelíbí htmlku...)
Owner

HTML je to AFAIK naprosto jedno, ostatně i v tom zmíněném base.html ty taby někde jsou… OK, budou dvě mezery.

HTML je to AFAIK naprosto jedno, ostatně i v tom zmíněném `base.html` ty taby někde jsou… OK, budou dvě mezery.
ledoian added 1 commit 2 years ago
ledoian changed title from editorconfig to editorconfig (#1011) 2 years ago
ledoian added 1 commit 2 years ago
Poster
Owner

Už mě nic nenapadá a funguje mi to :)

Už mě nic nenapadá a funguje mi to :)
zelvuska merged commit 577860985a into master 2 years ago
zelvuska deleted branch editorconfig 2 years ago
The pull request has been merged as 577860985a.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.