Bug/NFTPAR 1070 horizontal paddings #84
Conversation
…on-search Bugfix/nftpar 1002 collection search
packages/apps/src/apps.scss
Outdated
| font-weight: 500; | ||
| font-size: 16px; | ||
| margin: 4px 0; | ||
| padding: 8px 16px; |
There was a problem hiding this comment.
please use gap for paddings
There was a problem hiding this comment.
погодь, я сначала задачки по сути сделаю, потом возьмемся за рефакторинг)
There was a problem hiding this comment.
Тут не так много, чтобы не порефачить. Весь новый код должен быть в новом стиле.
There was a problem hiding this comment.
Кость, это не новый код - это я просто перенесла кусок, потому что разбросаны были по файлу куски от одного и тоже и дубликат лежал
There was a problem hiding this comment.
Завел задачу на рефактор:
https://cryptousetech.atlassian.net/browse/NFTPAR-1085
давайте сразу поправим если видим, а то до полноценного рефактора мы еще не скоро доберемся, а так, пока мимо проходим - поправить бы чтоб в следующий раз глаза не резало
There was a problem hiding this comment.
поправлю, как закончу задачки. Кость, ты пока не отвлекайся на мои коммиты - это я мелко-задачки пошагово лью (упражняюсь со взаимодействием github с jira:) )
У меня, как готово будет к просмотру, я тебе маякну ;)
packages/apps/styles/global.scss
Outdated
| @@ -198,7 +198,7 @@ h4.ui.header { | |||
| height: 48px; | |||
|
|
|||
| & .menu > .item { | |||
| padding: 8px calc( (var(--gap) / 2) * 5) !important; | |||
| padding: 8px calc((var(--gap) / 2) * 5) !important; | |||
There was a problem hiding this comment.
please use gap for paddings
packages/apps/styles/global.scss
Outdated
| p { | ||
| margin: 0; | ||
| color:var(--modal-warning-text-color); | ||
| color: var(--modal-warning-text-color); | ||
| font-family: var(--font-roboto); | ||
| padding: 0.3rem 1.5rem 0 0 !important; |
packages/apps/src/AppContext.tsx
Outdated
|
|
||
| const AppCtx = React.createContext<IAppContext>({ previewButtonDisplayed: false, setPreviewButtonDisplayed: () => { } }); | ||
|
|
||
| function AppProvider({ children }: Props): React.ReactElement<Props> { |
There was a problem hiding this comment.
а что это в паддингах? лучше это в отдельной задаче на previewButton сделать, я найду таску, на тебя поставлю
No description provided.