Skip to content

chore(docs): use mc-navbar in docs and add skins in docs (#UIM-898) #847

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lskramarov
Copy link
Contributor

No description provided.

@lskramarov lskramarov requested a review from pimenovoleg as a code owner April 7, 2022 14:29
@lskramarov lskramarov changed the title chore(docs): use mc-navbar in docs and add skins in docs (#UIM-898) chore(docs): use mc-navbar in docs and add skins in docs (#UIM-898) WIP Apr 7, 2022
@PositiveJS
Copy link
Contributor

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

@PositiveJS
Copy link
Contributor

Preview docs changes for 46fac12 at https://positive-js.github.io/mosaic-previews/pr847-46fac12/

@lskramarov lskramarov requested a review from agalaktionov April 8, 2022 11:16
@lskramarov
Copy link
Contributor Author

@agalaktionov можно смотреть.
Так же осталось решить проблему с иконками + в навбаре обнаружил парочку багов, задачи закинул, следом будем фиксить.

@agalaktionov
Copy link
Contributor

нужные иконки добавлены в mosaic-icon https://github.com/positive-js/mosaic-icons/releases/tag/6.3.0

@agalaktionov
Copy link
Contributor

agalaktionov commented Apr 9, 2022

Цвет нужный не подставился в скин 2017
image

@agalaktionov
Copy link
Contributor

agalaktionov commented Apr 9, 2022

При выборе скина 2017 не меняются:

  • фон левого сайдбара
  • табы-якори справа
  • Компоненты:
    • Progress-Bar
    • Progress-Spinner
    • Tree (стрелка в скине 2022)
    • Validation (заголовки таблицы «плохо»\«хорошо», тексты ошибок к полям в цветах 2022)

@agalaktionov
Copy link
Contributor

Давай уберем кликабельность с мозаика, так как нам некуда пользователя отправлять при нажатии
image

@dmitry-dedukhin
Copy link
Contributor

Цвет нужный не подставился в скин 2017 image

Я заводил UIM-901

@lskramarov
Copy link
Contributor Author

Давай уберем кликабельность с мозаика, так как нам некуда пользователя отправлять при нажатии image

по гайду оно всегда кликабельно:
image

@lskramarov lskramarov changed the title chore(docs): use mc-navbar in docs and add skins in docs (#UIM-898) WIP chore(docs): use mc-navbar in docs and add skins in docs (#UIM-898) Apr 11, 2022
@sonarcloud
Copy link

sonarcloud bot commented Apr 11, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
58.3% 58.3% Duplication

@PositiveJS
Copy link
Contributor

Preview docs changes for 175b0df at https://positive-js.github.io/mosaic-previews/pr847-175b0df/

@mikeozornin
Copy link
Contributor

mikeozornin commented Apr 12, 2022

  1. Заголовок СКИН набран не тем кеглем.
    Должно быть: прописные regular с разрядкой (body caps в итоге)
    Сейчас: прописные medium без разрядки
    image

  2. При переключении скина переключается только цветовая тема, типографика остается на месте. Нужно переключать всю тему целиком. Если бы была разница в отступах, то и нужно и их было бы переключать.

  3. И давай фон левой панели ещё перестанем хардкодить hex-кодом, пока мы не разобрались с background-стилями, потом переедем на них.
    Светлая: Secondary-40
    Темная: Secondary-800

@lskramarov
Copy link
Contributor Author

  1. Заголовок СКИН набран не тем кеглем.
    Должно быть: прописные regular с разрядкой (body caps в итоге)
    Сейчас: прописные medium без разрядки
    image

на макетах нет никакого caps:
image

нужно править макеты..

  1. При переключении скина переключается только цветовая тема, типографика остается на месте. Нужно переключать всю тему целиком. Если бы была разница в отступах, то и нужно и их было бы переключать.

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

@mikeozornin
Copy link
Contributor

Я ещё в тот комментарий дописал новое, ты можешь пропустить, вот отдельно:
И давай фон левой панели ещё перестанем хардкодить hex-кодом, пока мы не разобрались с background-стилями, потом переедем на них.
Светлая: Secondary-40
Темная: Secondary-800

@mikeozornin
Copy link
Contributor

на макетах нет никакого caps:

Да, из-за нового скина в цеплине отлетел стайлгайд, из-за этого не видно (вообще стиля нет). Мы видимо макеты и спеки для нового скина будем класть отдельно, чтобы стайлгайд был прилинкован.

@lskramarov
Copy link
Contributor Author

lskramarov commented Apr 13, 2022

Я ещё в тот комментарий дописал новое, ты можешь пропустить, вот отдельно: И давай фон левой панели ещё перестанем хардкодить hex-кодом, пока мы не разобрались с background-стилями, потом переедем на них. Светлая: Secondary-40 Темная: Secondary-800

как ты понял, что там хардкод ? там вот так:
image

ну т.е. с какой целью мы сейчас меняем 700 на 800 ?

Давай наверно остановимся на навбаре и не будем сейчас менять, то что не планировали. По изменениям в доках можно завести задачу и накидывать туда пунктами все, что хотим переделать.

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