fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

Удалить jss #115

Closed krashaen closed 4 years ago

krashaen commented 5 years ago

Мы вроде решили, что уходим от jss? Но тут он везде остался, предлагаю выпилить) @Znack @in19farkt @chmnkh @NikitaRzm Что думаете?

Znack commented 5 years ago

Мне сложно что-то конкретное сказать, я только на классике работал. Буду ждать мнения остальных ребят :)

in19farkt commented 5 years ago

можно конечно выпилить. Но сейчас на стартер ките используется material-ui, он хорош по многим параметрам и многие ребята его уже хорошо знают. Если мы принимаем решение выпилить jss, то придется вместе с ним выпилить material-ui и все форм филды написанные на основе материаловских компонент. Нужно будет найти какую-то достойную замену материалу и чтобы там не было css-in-js.

Мы смотрели в сторону antd, но чето там как-то сыровато, их компонентами не возможно пользоваться с клавиатуры, эломентарно табом по контролам ходить нельзя. Тут мне кажется нужно хорошенько погуглить, может действительно есть что-то очень крутое с классическими стилями.

С другой стороны мы можем оставить jss для material-ui, а самим использовать scss. Они у себя не используют динамические стили, поэтому просадки по производительности не будет. И как минимум кастомизировать mui через jss очень удобно. Но если мы в свою очередь будем пользоваться scss, то у нас не будет доступа к mui-теме, а это значит нам в двух местах придется поддерживать тему, специально для mui и для scss.

NikitaRzm commented 5 years ago

material за счет этого становится хуже всего остального, когда начинает диктовать такие крупные движения в бандле и стеке и только по этому можно от него отказаться и от подобных библиотек, когда они несовместимы по стеку. Мы же не тянем компоненты Vue условно для закрытия функционала работая на react.

Если в стеке оставили scss -> pure css, то надо стараться чтоб компоненты не требовали поддержания альтернативных подходов - это либо инкапсуляция css-in-js  глубоко под капотом (что довольно сложно, по-моему), либо работали в нескольких режимах, либо предлагали ровно то, что подходит стеку.

Так что считаю, что вопрос плавно перетекает в замену ui-kit'а стартеркита :)

in19farkt commented 5 years ago

А давайте еще с такой стороны посмотрим, представим что у нас есть ui-kit библиотека с pure css. Что нам это даст:

На что нужно обратить внимание при выборе юи кита:

Это так, на вскидку, что пришло в голову. Возможно у кого-нибудь есть чем дополнить :)

NikitaRzm commented 5 years ago

Генерализую немного критерии в отрыве от css-tech:

in19farkt commented 4 years ago

Думаю можно эту ишу закрывать