Simplify amount implementation #1101
Comments
@nin-jin, не стесняйтесь нам создавать issue, если заметили проблему, а еще лучше PR. В любом случае, Спасибо за нелестную оценку! @nin-jin
Подскажите, где такой взяли? |
Нигде, разумеется, просто представил что бы мне пришлось написать, используя я ваши компоненты. Если что-то лишнее - скажите ,я поправлю статью. |
@nin-jin theme у нас проставляется один раз с помощью ThemeProvider, но так же можно и явно поставить через пропсы. в ожидаемом примере из статьи
вы их и не используете. А пример выглядит намного минималистичнее. Плюс, если дальше развивать тему в "Плохом примере" вы описываете
А вот по поводу size - вы предлагаете брать size от родителя внутри вложенного компонента? |
То есть мой компонент надо дополнительно усложнить, чтобы можно было получать размер и из пропсов и из контекста. Иначе он не будет вписываться в вашу экосистему, где можно и так и сяк.
Потому, что задание их через пропсы не имеет смысла. Когда один компонент собираешь из других компонент - незачем вручную прокидывать тему во вложенные компоненты.
Потому что там не реакт вовсе. Обычный html-шаблон.
Да, это тоже контекстно зависимая штука. Если я указал size для компонента, то ожидаю, что он увеличится со всем содержимым, а не только паддинги вокруг него. |
Не совсем так в вашем случае для темы ничего делать не нужно, она сама прорастет.
Тут я согласен. По имплементации как предлагаете прокидывать size? react контекстом? |
Это вы исходите из того, что на ваших компонентах будут делать лишь приложения, но не другие компоненты. Представьте, каково будет пользователю моих и ваших компонент вперемешку. В одних компонентах тему можно задать через пропсы, в других нельзя.
Ну да, контекстом. Или даже просто через css-zoom. |
https://habr.com/ru/post/491120/
@nin-jin
Указал нам на проблемы amount
Предлагает дойти до реализации в таком вот виде
Большинство нод в свое время было добавлено из за разности прозрачности отдельных элементов. Думаю щас большинство не актуально.
DIv в span (facepalm)
Давайте попробуем причесать
The text was updated successfully, but these errors were encountered: