search

positive-js / mosaic

Angular components ✨ 📦🐣 ✨
https://mosaic.ptsecurity.com
MIT License
143 stars 40 forks source link

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

Open lskramarov opened 2 years ago

PositiveJS commented 2 years ago

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

PositiveJS commented 2 years ago

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

lskramarov commented 2 years ago

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

agalaktionov commented 2 years ago

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

agalaktionov commented 2 years ago

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

image
agalaktionov commented 2 years ago

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

agalaktionov commented 2 years ago

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

image
dmitry-dedukhin commented 2 years ago

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

Я заводил UIM-901

lskramarov commented 2 years ago

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

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

sonarcloud[bot] commented 2 years ago

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 commented 2 years ago

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

mikeozornin commented 2 years ago

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

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

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

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

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

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

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

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

mikeozornin commented 2 years ago

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

mikeozornin commented 2 years ago

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

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

lskramarov commented 2 years ago

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

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

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

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