shri-2015-org / 666

666 team's chat =>
http://anonym.club
4 stars 3 forks source link

Layout refactoring #42

Closed exah closed 9 years ago

deemidroll commented 9 years ago

Почему RoomInput, а не, скажем, MessageInput? Я вот теперь думаю, как переносить такую структуру на react-native, там нет особого смысла отделять стили. Вообще здорово, +1

tempname11 commented 9 years ago

А как насчет сразу перенести client/components -> components? Это будет иметь смысл для server-side рендеров, если мы до них доберемся. Ну и просто удобнее, мне кажется.

pofigizm commented 9 years ago

Для серверного рендеринга не только файлы компонент понадобятся, так что то что компоненты внутри client - мне кажется нормальным.

tempname11 commented 9 years ago

@Pofigizm Если какой-то еще код будет общим для клиента и сервера, не лучше ли его явно вынести в common?

pofigizm commented 9 years ago

Одно дело общий код (модули, либы и пр ), а другое дело пререндер (как я понимаю берется весь браузерный код и рендерится в имитации браузера)

@tempname11

tempname11 commented 9 years ago

@Pofigizm серьезно? А я думал что серверный рендер просто генерит html-строку без всякой претензии на имитацию браузера. Надо еще почитать про это.

mightyaleksey commented 9 years ago

На мой взгляд лучше весь код по возможности делать универсальным (компоненты + сторонние модули). Под браузер можно отвести отдельный файл, который будет делать начальный рендер и ставить полифилы. На сервере будут роуты, логика и т.п.

Впрочем дело вкуса :)

mightyaleksey commented 9 years ago

Ну и да, к слову о терминологии: https://medium.com/@mjackson/universal-javascript-4761051b7ae9

tempname11 commented 9 years ago

@sullenor да, универсальность это было бы здорово.

pofigizm commented 9 years ago

тогда получается в корне лежат два файла server и client, а все остальное по своим папкам: modules, components etc.

mightyaleksey commented 9 years ago

Я бы под сервер отдельную папку отвел. Так как там будет не только код сервера, но еще и роуты описаны, хранилище. А когда дойдем до деплоя, то нужно будет еще и масштабировать его (кластеры / рекластеры).

exah commented 9 years ago

Так что, я же делаю merge? где ещё +1?)

tempname11 commented 9 years ago

Я хотел почитать коммиты и сделать плюс-один, но закопался со своей веткой и уже не успею) спать надо)

В крайнем случае думаю не проблема будет завтра замержить очно.