@react_js

Страница 3021 из 5115
Andrey
26.01.2018
21:02:12
Мне плевать что в className написано.

Котяй Негодяй
26.01.2018
21:02:39
+ вёрстка отдаётся на сторону.
Ну, у нас дизайнер шарит в css-in-js.

Andrey
26.01.2018
21:02:56
Вот, в этом разница.

Google
Andrey
26.01.2018
21:03:18
Если всё нормально, то без проблем. К нашему проекту css-in-js не применимо.

Котяй Негодяй
26.01.2018
21:03:20
Но он всё равно делает в css.

Мы слизываем стили с зеплина.

Вообще нормально.

Andrey
26.01.2018
21:04:16
Дмитрий
26.01.2018
21:04:17
А в SC разве можно? В styled-jss точно нет. Смотрю доку SC, и пока не нашёл.
Какая теперь разница когда есть новый context api)

Котяй Негодяй
26.01.2018
21:04:33
dima
26.01.2018
21:05:43
Ну, у нас дизайнер шарит в css-in-js.
а дизайнер причем ? он же сказал что верстают гна стороне

Andrew
26.01.2018
21:06:32
zeplin.io
вообще топ тема

очень херово когда верстку дают на аутсорс

Andrey
26.01.2018
21:07:24
Ну, как есть.

Andrew
26.01.2018
21:08:57
проще своего научить а еще лучше что б дизайнер это умел делать

Google
Andrew
26.01.2018
21:09:03
это прям вообще супер

Andrey
26.01.2018
21:14:53
Короче говоря, выбор инструментов зависит коренным образом от окружения, в котором проект находится.

Andrew
26.01.2018
21:17:07
верстать дизайнеры ? ты че
пусть хоть понимают как их дизайн будет в браузере жить

я уже не говорю о ux

dima
26.01.2018
21:17:38
пусть хоть понимают как их дизайн будет в браузере жить
их это не должно волновать. А верстка и ux причем ?

Andrey
26.01.2018
21:18:00
Ага, не должно, конечно.

А потом получаем паралакс, которой невозможно нормально реализовать.

Andrey
26.01.2018
21:18:41
Или ещё какую-нибудь хреновину.

dima
26.01.2018
21:19:03
Ага, не должно, конечно.
я что то о UX сказал ? их не волнуют тонкости. А если рисуют заоблачный функционал то такое конечно нафиг не нужно

Andrew
26.01.2018
21:19:08
или мол у тебя тут шрифты не так рендерятся

Andrey
26.01.2018
21:19:31
Andrew
26.01.2018
21:19:45
а не когда тебе надо пачку инлайн стилей на лету менять что их херня чето там делала

Ruslan ?
26.01.2018
21:23:26
проще своего научить а еще лучше что б дизайнер это умел делать
Научить нейронную сеть верстать по картинке и всё https://pikabu.ru/story/neyronnaya_set_kotoraya_verstaet_saytyi_po_kartinke_5629901

Дмитрий
26.01.2018
21:24:28
Default
26.01.2018
21:24:29
Идеально эт когда десигнеры сразу фигачат компоненты и из них собирают странички

А верстают они при этом или нет при таком подходе дело десятое

Andrey
26.01.2018
21:25:14
Я бы в withTheme хок вытащил и было бы норм.

Google
Default
26.01.2018
21:25:27
Самый большой геморрой эт стопицот разных свистоперделок

Andrey
26.01.2018
21:25:33
Мне не нравятся рендер функции.

Самый большой геморрой эт стопицот разных свистоперделок
Если их кто-то реализовал, то без проблем)

Дмитрий
26.01.2018
21:26:08
Я бы в withTheme хок вытащил и было бы норм.
Тут нечего вытаскивать в хок, совсем

Andrey
26.01.2018
21:26:27
Тут нечего вытаскивать в хок, совсем
Меньше кода, проще читается.

Дмитрий
26.01.2018
21:26:39
Тут нечего вытаскивать в хок, совсем

Default
26.01.2018
21:26:41
Если их кто-то реализовал, то без проблем)
Да в любом случае не ок. Когда у тебя стопицот разных кнопок это дико напрягает

Andrey
26.01.2018
21:26:44
Но ладно, это вкусовщина.

Дмитрий
26.01.2018
21:27:10
Я не говорю меньше больше, я говорю что тут нет хока

Компонент Main получает свои пропсы и так же забирает через рендер проп ещё и определенный контекст

Andrey
26.01.2018
21:28:29
Я понимаю, но вот эту обвязку с прокидыванием темы я бы из компонента убрал и оставил бы просто содержимое рендер-функции.

Дмитрий
26.01.2018
21:28:53
А пропсы сверху откуда возьмутся?))

Andrey
26.01.2018
21:29:02
withTheme

Дмитрий
26.01.2018
21:29:29
Ты не понял, пропсы сверху это и есть сами пропсы компонента

Здесь уже обёрнуто всё

Andrey
26.01.2018
21:29:43
Я понял.

Дмитрий
26.01.2018
21:29:54
В качестве withTheme — Theme

Andrey
26.01.2018
21:30:14
Понимаю. И я считаю, что это не надо писать в компоненте.

Дмитрий
26.01.2018
21:31:10
Помимо всего прочего этот компонент атомарен и без темы лишён смысла и хорошего вывода типов

Andrey
26.01.2018
21:31:47
Эээ, походу ты не понимаешь меня.

Google
Дмитрий
26.01.2018
21:32:26
Ну покажи как бы ты вынес, что осталось бы в Main)

Andrey
26.01.2018
21:33:48
Ну покажи как бы ты вынес, что осталось бы в Main)
Написал бы хок withTheme, который так же оборачивает переданный компонент A компонентом Theme и передаёт в A пропсом theme.

И потом бы просто его обернул в импорте.

Это, конечно, вкусовщина, но зачем в компоненте писать лишнее?

Дмитрий
26.01.2018
21:37:53
А, теперь вижу, действительно, два компонента никогда не лишние)

Andrey
26.01.2018
21:38:26
Эм?

Дмитрий
26.01.2018
21:40:06
Был один компонент с одним пропсом, стало двое с двумя

Charles
26.01.2018
21:41:11
ты скажи ему что ник слишком длинный. ну рилли

Andrey
26.01.2018
21:41:11
Эм... А какие 2 компонента стали?

Admin
ERROR: S client not available

Дмитрий
26.01.2018
21:41:31
Andrey
26.01.2018
21:41:48
Дмитрий
26.01.2018
21:42:32
ты скажи ему что ник слишком длинный. ну рилли
Могу вернуть старый local variables referenced from the lambda expression must be final

Andrey
26.01.2018
21:43:00
Не вижу в этом проблемы. Если код от чего-то становится чище, то надо делать.

Charles
26.01.2018
21:43:15
+

Дмитрий
26.01.2018
21:44:31
Я не вижу в этом чистоты. Просто попробуй написать, не рассказать на словах, а реально выписать рядом свой вариант и сравнить

Помимо всего прочего у тебя в типе пропсов компонента появляется сам контекст целиком

Дмитрий
26.01.2018
21:50:34
Ты заметил что даже не смотря на многоточия на месте самого компонента у тебя всё равно больше кода?

Google
Пг
26.01.2018
21:50:43
Вообще, scss адекватнее в плане размера кода, возможности вынести все в отдельные файлы\файл, дабы параллельно загружать и парсить отдельно от js-ки с куда большей скоростью

))0)

Andrew
26.01.2018
21:51:06
Не

Andrey
26.01.2018
21:51:07
Ты заметил что даже не смотря на многоточия на месте самого компонента у тебя всё равно больше кода?
Дык без проблем. Я посмотрел, что там есть тема и всё, больше туда не смотрю.

Человек
26.01.2018
21:51:16
подскажите есть массив cчетов и в нем внутри показания по приборам, пользователь может их обновлять. как мне правильно организовать state

Котяй Негодяй
26.01.2018
21:51:23
В данном случае, theme — это и есть контекст?

Andrey
26.01.2018
21:51:24
И меня хоки не волнуют.

Andrey
26.01.2018
21:52:17
А количество кода и размер сущности - это совершенно разные вещи.

Дмитрий
26.01.2018
21:52:20
Принимается непосредственно из источника контекстных данных

Пг
26.01.2018
21:53:16
не надо вбросов
ну так правда же, еще и раздельное кеширование в придачу, что позволяет обновлять у клиента стили независимо от js-ки

Andrey
26.01.2018
21:55:12
@ZeroBias А если ещё что-то из контекста понадобится принять, то что делать будешь?

Так же? 2 рендер функции и 2 декоратора-объект?

В случае хоков у тебя проблемы никакой. Хоть 10 переменных из контекста. Компонент будет таким же лаконичным.

Дмитрий
26.01.2018
21:57:44
А кто сказал что контекст будет всего один?)

Ладно, я сдаюсь, я не знаю как ещё пояснить почему лишние, совершенно нетипизируемые пропсы, ещё один компонент, чей тип определяется неявно аж от двух дженериков, не решающий никакой осмысленной задачи кроме эстетической, и сводящий на ноль все бонусы нового апи непосредственно — это не очень

Дмитрий
26.01.2018
22:02:53
Потому что они вносят в тип компонента тип контекста. Если быть точным, то этот подход вносит в тип компонента все контексты во всех вариациях, которые его угораздило тут использовать

И возвращает назад проблему конфликтов имён контекстов

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