VKCOM / VKUI

VKUI – это набор React-компонентов, с помощью которых можно создавать интерфейсы, внешне неотличимые от наших iOS и Android приложений.
https://vkcom.github.io/VKUI/
MIT License
1.04k stars 185 forks source link

fix(ImageBase): fix bug with css selectors of ImageBase #8004

Open EldarMuhamethanov opened 5 days ago

EldarMuhamethanov commented 5 days ago

Описание

Сейчас при отображении в PanelHeader компонента Avatar в after/before добавляется margin по бокам. Есть ощущение, что эти отступы не нужны. В общем, погуляя по истории git я нашел pull request в котором был добавлен отступ. Это было сделано аж в 2020 году, так я и не понял, какая тогда была версия VKUI, но точно меньше 3.12.4(это последняя версия, которую можно посмотреть через сайт). Проблема в том, что все также непонятно конкретно зачем это было сделано. Там конечно есть упоминание этой задачи, но связать это с изменениями я не понял как, возможно был запрос от дизайна когда-то тогда, либо автор сам решил, что отступ нужен. Решил, что можно доработать селектор, добавив ">", чтобы только прямому потомку добавлялся отступ. А при использовании Avatar с оберткой в виде PanelHeaderButton отступ не будет добавляться.

Изменения

Поправил селектор, добавив ">", чтобы только прямому потомку добавлялся отступ

Для DesignReview

Чтобы посмотреть отображение Avatar в PanelHeader, нужно перейти на страницу PanelHeader и выключить настройку hasCustomPanelHeaderAfter. Просьба посмотреть отображение Avatar в after PanelHeader на разных платформах и размерах экранов.

Release notes

Исправления

github-actions[bot] commented 5 days ago

👀 Docs deployed

Commit 414652f95e855ad7760cc9c4f3f4e4956e1d0cc8

codecov[bot] commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.46%. Comparing base (027d01d) to head (414652f). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #8004 +/- ## ========================================== - Coverage 95.47% 95.46% -0.01% ========================================== Files 380 380 Lines 11325 11324 -1 Branches 3777 3777 ========================================== - Hits 10812 10811 -1 Misses 513 513 ``` | [Flag](https://app.codecov.io/gh/VKCOM/VKUI/pull/8004/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=VKCOM) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/VKCOM/VKUI/pull/8004/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=VKCOM) | `95.46% <ø> (-0.01%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=VKCOM#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.