@react_js

Страница 4133 из 5115
AlexAnder
09.06.2018
07:58:01


т.к. элементов мало то он в одну линию фигачит а потом добавляешь пару элементов и он все раскидывает на новую строку

Anton
09.06.2018
07:58:22
@ZeroBias может тебе и не такое рассказать о том, как пользователи будут юзать библиотеку
У меня таких историй наверное более будет. Некоторые совсем печальные.

Google
Max
09.06.2018
07:58:47
@sergeysova export const CustomFieldsForm = compose<InnerProps, OuterProps>( FormValidator(validationRules), Form )(CustomFieldsFormFields) export const CustomFieldsForm = compose<InnerProps, OuterProps>(FormValidator(validationRules), Form)( CustomFieldsFormFields, )

композы портит

Max
09.06.2018
08:00:13
ну вот я те показал "до" и "после"

Сергей
09.06.2018
08:00:30
а

хреново

Max
09.06.2018
08:01:11


вот пример днищенский с переносом дженерика

Таймураз
09.06.2018
08:01:57
Второе- после приттиера?

Max
09.06.2018
08:01:58
да

Таймураз
09.06.2018
08:02:08
Ебани им ишью

Max
09.06.2018
08:02:10
тут конечно тип перетащить можно в декларацию

Google
AlexAnder
09.06.2018
08:02:27
Dict<ValidationError[]>[]

Max
09.06.2018
08:02:41
<a href="" onClick={this.handleClick}> View </a>

Andrey
09.06.2018
08:02:50
Max
09.06.2018
08:02:52
разворачивание в мультилайн non-block элементов

Таймураз
09.06.2018
08:03:08
<a href="" onClick={this.handleClick}> View </a>
Хз Вот с этим я согласен

Строка- тоже компонент

Ну и так чуть более читаемо

Artyom
09.06.2018
08:04:40


Max
09.06.2018
08:05:02
<tbody> {value.map((row, index) => <tr key={index}> {headers.map((header) => ( <td key={header.key}> {row[header.key]} </td> ))} </tr> )} </tbody>

вот читаемый компонент

Сергей
09.06.2018
08:05:08
первое попроще

Max
09.06.2018
08:05:15
<tbody> {value.map((row, index) => ( <tr key={index}>{headers.map((header) => <td key={header.key}>{row[header.key]}</td>)}</tr> ))} </tbody>

вот ебанина после преттиера

Artyom
09.06.2018
08:05:53
первое попроще
Меня напрягает, что искать что-то сложно, т.к. из-за запятых (которых подчти не видно беглым взглядом) все размывается

Max
09.06.2018
08:07:00
кстати тернарки только мы так пишем? const ariaProps = ariaLabel ? { 'aria-label': ariaLabel } : { 'aria-hidden': true }

Max
09.06.2018
08:07:16
const ariaProps = ariaLabel ? { 'aria-label': ariaLabel } : { 'aria-hidden': true }

после преттиера

Google
Max
09.06.2018
08:07:54
не, он на 95% нормально отрабатывает (в смысле я готов с этим жить)

но иногда выдает такую ебанину

я его 3 раза ставил на проект

потом с матами убирали (

Yumi
09.06.2018
08:08:18
вот ебанина после преттиера
Много такого, но преимущества преттиера в том что он делает код консистентным и позволяет забить на форматирования кода и писать код, очень много времени экономит.

Таймураз
09.06.2018
08:08:22
Yumi
09.06.2018
08:08:47
он не делает код консистентным
В плане форматирования нет?

Max
09.06.2018
08:08:47
консистентность это с точки зрения преттиера

Сергей
09.06.2018
08:08:50
В плане форматирования нет?
иногда так, иногда так это хрень

Max
09.06.2018
08:09:04
а с точки зрения проекта код выглядит по-разному

который должен выглядеть одинаково

export const FieldMappingFormModal = compose( ModalForm, Form, )(FieldMappingFormFieldSet)

вот у нас принято такое использование композа

потому что его может быть от 1 до 10 строк

Max
09.06.2018
08:10:07
преттиер делает в одном месте export const FieldMappingFormModal = compose<any, any>(ModalForm, Form)(FieldMappingFormFieldSet)

Google
Max
09.06.2018
08:10:24
в другом месте ```export const CustomFieldsForm = compose<InnerProps, OuterProps>(FormValidator(validationRules), Form)( CustomFieldsFormFields, )```

что не выглядит консистентно

и ухудшает чтение кода проекта

Max
09.06.2018
08:10:49
нет конечно

Таймураз
09.06.2018
08:11:06
А, я понял о чем ты

Ну когда он так ебано форматирует, я разбиваю такие декларации на несколько этапов

Max
09.06.2018
08:11:42


Admin
ERROR: S client not available

Max
09.06.2018
08:11:51
вот выше кидал пример с <a>

Таймураз
09.06.2018
08:11:57
Сначала hoc функцию пишу, потом уже оборачиваю компонент

Max
09.06.2018
08:12:08
почему тут свернул а там развернул? это и многое другое в журнале "а хуй его знает"





вот более явно?

нахуя тут сворачивать див а <a> там разворачивать?

где тут консистентность?

Yumi
09.06.2018
08:13:01
преттиер делает в одном месте export const FieldMappingFormModal = compose<any, any>(ModalForm, Form)(FieldMappingFormFieldSet)
Хм. Я в двух проектах использую prettier через eslint, он не убирает новые линии в композициях, но в онлайн repl действительно работает, как ты и говоришь.

Таймураз
09.06.2018
08:13:32
нахуя тут сворачивать див а <a> там разворачивать?
Когда компонент большой, тогда лучше разоворачивать А если у тебя маленькая вложенность- лучше ее уменьшить

Google
Max
09.06.2018
08:13:51


ОХУЕТЬ ОГРОМНЫЙ КОМПОНЕНТ

(простите)

Таймураз
09.06.2018
08:14:24
то есть return () и внутри этот компонент?

И он его на 3 строки разбивает?

Max
09.06.2018
08:14:36
нет, это внутри html-a

как и на скрине выше с дивом внутри layer-a

Таймураз
09.06.2018
08:15:14
Ну если там есть еще 3+ строк кода- тогда уже не так спорно

Сергей
09.06.2018
08:16:05
Таймураз
09.06.2018
08:16:06
Имхо когда приттиер что-то делает ебано- легче переписать код

Max
09.06.2018
08:16:17
Имхо когда приттиер что-то делает ебано- легче переписать код
ну вот делать мне нехуй хороший код переписывать

Таймураз
09.06.2018
08:16:24
Он еще и более явный станет Вот пример с матрицей- ебанутый, согласен

Max
09.06.2018
08:16:25
потому что тулза считает не так

Сергей
09.06.2018
08:16:26
бля

Таймураз
09.06.2018
08:16:28
И еще несколько примеров

Сергей
09.06.2018
08:16:31
я лолкаю больше чем думаю

Max
09.06.2018
08:16:38
щас обновлю преттииер

может что-то починили

в принципе можно // prettier-ignore пихать иногда

Страница 4133 из 5115