хочется раз настроить и забыть

еще практикуется ставить пробел между скобками и содержимым в jsx

например ={ this.state.email }

но это тоже мелочи конечно

так типа?

например ={ this.state.email }

а понял

но это тоже мелочи конечно

вот о них как раз и спрашиваю )

ну вот, я ж говорю, щас научат) пробелы внутри скобок и тп) берите airbnb и все, остально внутри команды обсуждается за 5 минут и го

согласен) все так и делают обычно

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

ну ок пока оставлю так, а там уже договоримся

спасибо

standard ещё норм, к вебсторму прикручивается последнему

зачем обсуждать в команде стандарт)

выбрали и все

у airbnb шлак полный eslint

standard лучше + свои кастомные рулы

у airbnb шлак полный eslint

Рубисты, сэр.

да рубистов и питонистов сразу за версту по коду видно, когда в JS лезут

у airbnb шлак полный eslint

почему?

почему?

лично мне он не нравится, отсутствие точек с запятыми, были еще различные косяки, JS получается не такой теплый и ламповый. Я щас не слежу за ними активно

в airbnb вроде наоборот правило: "ставить точки с запятыми"

в airbnb вроде наоборот правило: "ставить точки с запятыми"

может изменили что то?

в airbnb вроде наоборот правило: "ставить точки с запятыми"

вот-вот

ну не суть, просто имхо: "одинаково" иногда важнее чем "лучше"

вообще все готовые пресеты это как create-react-app, хорошо подойдет что то потестить, или для очень мелкого проекта, но когда работа от месяца и более, стоит сделать кастом

https://github.com/felixge/node-style-guide вот такой больше по душе как то, описаны самые основные вещи, нигде не перегибается палка, более гибко уже сам доделываешь

я даже свой конфиг сделал под себя https://github.com/vyorkin-personal/eslint-config-vyorkin но я не пользуюсь им, тк работаю на работе, а там надо чтобы одинаково) только для своих каких-то экспериментов и все, но по-моему это упоротость все

лично мне он не нравится, отсутствие точек с запятыми, были еще различные косяки, JS получается не такой теплый и ламповый. Я щас не слежу за ними активно

ващет наоборот на ASI в standard надеются

у airbnb шлак полный eslint

Почему?

у airbnb шлак полный eslint

почему?

Кто тестил компонеты с помощью Jest?

Кто тестил компонеты с помощью Jest?

Я в одном из пет-проектов начал переходить с mocha/chai/istanbul на jest. Хотя еще мало тестов перенес. Что-то конкретное интересует?

Кто тестил компонеты с помощью Jest?

Я тестил

Да мне тут адаптивность компонента затестить надо.

Но я уже, вроде, понял как.

Ну что, уже можно на второй вебпак переезжать?

Ну что, уже можно на второй вебпак переезжать?

Угу. Даже дока по миграции вполне норм

ну вот читаю

вроде не так много

Можете, плиз, отревьюить код моего первого приложения на реакте?

Код херня, надо всё переписать

Можете, плиз, отревьюить код моего первого приложения на реакте?

сцылкой кинь на гитхаб :)

https://github.com/shashkovdanil/reacttodoapp.io

а нахрена расширение файлам ставишь Jsx? охота с вебпаком дополнительно танцевать? :)

В смысле?

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

Там я так понимаю стэйдж 0 подключить надо для этого?

чего 0 то сразу? стрелочники вроде давно уже в ходу

много где нативно поддерживаются

у тебя нода какой версии?

Тогда методы внутри класса handleClick = () => {} так записывать?

ну попробуй, да

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

Ну я так делал, у меня почему то ошибка падала

Указывала она на такую запись метода

https://github.com/shashkovdanil/reacttodoapp.io/blob/master/src/App.jsx вот там список тудух я бы вынес в отдельный компонент, вместе с мапой

Ну да, я завтра разобью ещё на компоненты и чтоб потом в index.js только рендер был

даже так, тудуху в отедльный компонент, и мапу в отдельный компонент

А сохранение в локал сторэдж нормально сделано?

ну тут есть 2 подхода

лоакалсторадж хранит по сути пары ключ-строка, и можно хранить все в одном ключе, либо как ты, раскидывать по разным. я пока однозначно не скажу что лучше

Ну я сделал по одному чтобы удалять можно было

ну то что префикс есть, это уже хорошо :)

а вот такой момент - произвольно менять местами тудухи если бы понадобилось, как бы ты этот момент реализовал?

ERROR: S client not available

Пока даже не думал об этом

вот ты меняешь статус или меняешь тудуху, а в сторадж не отражаешь изменения :)

Упс, вот это не проверил)

Завтра пофикшу

вот это название не отражает суть имхо cachedTodo()

я бы назвал это дело storedTodos()

все же я бы сделал на 1 ключе хранилище, т.к. если я что-то еще буду хранить в сторадже, или даже много чего еще, то сканировать весь сторадж мне как-то не улыбается.

я бы сделал 1 функцию sync2storage() и через нее бы сохранял везде, где требуется, не дублируя логику

в общем моё мнение, для первого раза не дурно, но совершенству нет пределов :)

Ребят

Посоветуйте пожалуйста react table чтобы на 1000+ записей сортировка и поиск шустры были

Есть Facebook react table, но там последние коммиты пол года назад :(

fixed-data-table-2, react-virtualized, свое, в blueprintjs еще вроде виртуальный грид кстати

спасибо

А как скинуть ссылку на эту группу другу?

https://t.me/react_js

О, нашел.

https://github.com/shashkovdanil/reacttodoapp.io

Однострочные компоненты можно выражать как чистые функции без обвязок в класс https://facebook.github.io/react/docs/components-and-props.html Вот такие веши можно делать проще https://github.com/shashkovdanil/reacttodoapp.io/blob/master/src/components/Form.jsx#L8 просто пишем метод класса ` constructor() { .... } handleClick = () => {...}; ~ Подумай над отделением вью части от бизнес логики, это может быть выражено простейшим классом Todos, у которого будут методы удаления добавления, в которых ты будешь оперировать с local storage и с массивом

А webpack 2 оказывается норм, никаких осложнений

Однострочные компоненты можно выражать как чистые функции без обвязок в класс https://facebook.github.io/react/docs/components-and-props.html Вот такие веши можно делать проще https://github.com/shashkovdanil/reacttodoapp.io/blob/master/src/components/Form.jsx#L8 просто пишем метод класса ` constructor() { .... } handleClick = () => {...}; ~ Подумай над отделением вью части от бизнес логики, это может быть выражено простейшим классом Todos, у которого будут методы удаления добавления, в которых ты будешь оперировать с local storage и с массивом

Не однострочные, а без стейта

да

А webpack 2 оказывается норм, никаких осложнений

у меня не все лоадеры и не все плагины работают с последним вебпаком, например pug-loader, WatchIgnorePlugin, source-map-loader, etc хотя про WatchIgnorePlugin я не знаю зачем он нужен есть же watchOptions.ignored, котрый использует anymatch

Ребят, у меня 2 вопроса. Один нубский по реакту, второй нубский не по реакту ) Вот прям не могу уснуть.

Есть тут кто-то сейчас?

Вижу что есть )

Тута, тута.

Короче первый такой — зачем использовать пропсы, если их нельзя динамически изменять? Не проще везде тыкать стейты просто?

А как ты собираешься что-либо передавать компонету?

А стейт нельзя передавать?

а, ну понял вроде

А тут подробно вот: http://stackoverflow.com/a/32186324/3818282 Хороший ответ

Можно. Но ты его передаёшь через пропсы. И если технически придираться, то не его, а его копию, тогда уж.