search

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

95 show selected filters #101

Closed chmnkh closed 5 years ago

in19farkt commented 5 years ago

@chmnkh статус всё еще wip?

chmnkh commented 5 years ago

@in19farkt я думал дождаться мержа тестов хэлперов, дописать тут тесты для добавленных хэлперов и открыть, чтобы не забыть о тестах

chmnkh commented 5 years ago

не вип!

in19farkt commented 5 years ago

@chmnkh а можешь сюда тоже скрин прикрепить? :)

chmnkh commented 5 years ago

image

image

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

мне кажется это норм, но если кто-то вдруг прямо считает что надо строго определенный порядок задавать, то дискасс

Znack commented 5 years ago

Да, порядок фильтров пофиг, точно можно не заморачиваться

in19farkt commented 5 years ago

Единственное, мне кажется это нужно под кнопками отобразить. Таким образом мы избежим сдвига интерактивных элементов (инпут/кнопки).

chmnkh commented 5 years ago

Единственное, мне кажется это нужно под кнопками отобразить. Таким образом мы избежим сдвига интерактивных элементов (инпут/кнопки).

не очень согласен, мне кажется как сейчас аккуратнее выглядит: image

если сместить фильтры вниз, то будет как бы вперемешку конфиг для поиска (коими фильтры являются) + результаты поиска, и вообще много буков будет в куче, а сейчас текст как-то разбавляется элементами хотя бы. а сдвиг мне кажется вообще не критичен, я например даже не замечал, пока ты не сказал)) тут какой-то рамс с центрированием правда (фильтры из-за маргинов правых не совсем по центру получается, что заметно, надо это разрулить)

как ты предлагаешь: image

in19farkt commented 5 years ago

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

chmnkh commented 5 years ago

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