Skip to content

fix(common): removing unnecessary borders and fixing popover's background (#UIM-396) #449

Conversation

Margar1ta
Copy link
Contributor

@Margar1ta Margar1ta commented Apr 6, 2020

Removing unnecessary borders from sidepanel's, popover's and modal's footers

Popover background is fixed, popover component theme is removed and replaced by popup-common theme

Removing unnecessary borders from sidepanel's, popover's and modal's footers
@PositiveJS
Copy link
Contributor

Preview docs changes for 2457bbe at https://positive-js.github.io/mosaic-previews/pr449-2457bbe/

@Margar1ta Margar1ta changed the title fix(common): removing unnecessary borders fix(common): removing unnecessary borders (#UIM-396) Apr 6, 2020
@pimenovoleg pimenovoleg requested a review from mikeozornin April 6, 2020 15:11
Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Точно Антон сказал, что так ок?

В светлой теме все ок. Что меня смутило в темной:

  1. Появление какого-то подвала у поповера:

image

  1. Пропадание линии в подвале у модалки и сайдпанели, но линия в хедере поповера. Так ок?

@Margar1ta
Copy link
Contributor Author

Margar1ta commented Apr 6, 2020

Точно Антон сказал, что так ок?

В светлой теме все ок. Что меня смутило в темной:

  1. Появление какого-то подвала у поповера:
image
  1. Пропадание линии в подвале у модалки и сайдпанели, но линия в хедере поповера. Так ок?

Антон сказал, что бордер у футера в modal, popover и sidepanel лишний и его нужно убрать, что я и сделала. В макетах светлой темы этих бордеров действительно нет.

  1. Подвал у поповера есть сейчас в ветке master. Думаю это еще один баг, появившийся после влития токенов, который мы с Антоном не заметили. Я добавлю его в задачу.

  2. Прости, не проверила темную тему. Если чесно не совсем понимаю тогда, как сверстать на обе темы: получается в одной есть границы (в макете они реализованы как shadow а не border), а в другой этих границ нет:
    я могу сделать эти границы прозрачними в светлой теме и оставить в темной теме, как сейчас в ветке мастер, так пойдет?

@PositiveJS
Copy link
Contributor

Preview docs changes for 4965625 at https://positive-js.github.io/mosaic-previews/pr449-4965625/

Popover background is fixed, popover component theme is removed and replaced by popup-common theme
@PositiveJS
Copy link
Contributor

Preview docs changes for c71b41e at https://positive-js.github.io/mosaic-previews/pr449-c71b41e/

@Margar1ta Margar1ta changed the title fix(common): removing unnecessary borders (#UIM-396) fix(common): removing unnecessary borders and fixing popover's background (#UIM-396) Apr 9, 2020
@pimenovoleg pimenovoleg requested a review from agalaktionov April 9, 2020 14:34
@agalaktionov
Copy link
Contributor

В светлой теме заменить цвет нижних бордеров в хедерах в modal, popover и sidepanel на #cccccc

@PositiveJS
Copy link
Contributor

Preview docs changes for 05ebb9c at https://positive-js.github.io/mosaic-previews/pr449-05ebb9c/

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@PositiveJS
Copy link
Contributor

Preview docs changes for f274b7f at https://positive-js.github.io/mosaic-previews/pr449-f274b7f/

@pimenovoleg pimenovoleg merged commit 91edafd into positive-js:master Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants