Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(doc): now we support v4 demos #1595 #1634

Merged
merged 12 commits into from
May 16, 2024

Conversation

@ZurabDev ZurabDev added this to the 4.3.0 milestone Apr 22, 2024
@ZurabDev ZurabDev self-assigned this Apr 22, 2024
Copy link

github-actions bot commented Apr 22, 2024

Visit the preview URL for this PR (updated for commit cfee4ae):

https://prizm-v4--pr1634-feat-support-v4-priv-ltv73ew6.web.app

(expires Sun, 19 May 2024 13:06:53 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7c62ed8dbabf5e2d6b2084ca9e107cc206d30dbd

Copy link
Collaborator

@ickisIckis ickisIckis left a comment

Choose a reason for hiding this comment

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

LGTM

@alexhawkins94
Copy link

при переходе в пример base у Пагинтора ошибка на SB
из других примеров тоже
image

@ZurabDev
Copy link
Contributor Author

при переходе в пример base у Пагинтора ошибка на SB из других примеров тоже image

Готво

@ZurabDev ZurabDev added availability:reviewed Ready to test and removed state:reopen labels Apr 25, 2024
Copy link

@alexhawkins94 alexhawkins94 left a comment

Choose a reason for hiding this comment

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

ошибки в СБ при переходе в примеры, примеры не грузятся
input number
listing item
counter
prizm-doc-documentation
confirm dialog
dialog
sidebar
radiobutton
side menu
skeleton
slider
spinner
switcher
table
toast
togle
zoom control
prizm data list

нет кнопки для перехода на СБ
navigation menu
navigation

@ZurabDev
Copy link
Contributor Author

ошибки в СБ при переходе в примеры, примеры не грузятся listing item counter prizm-doc-documentation confirm dialog dialog sidebar radiobutton side menu skeleton slider spinner switcher table toast togle zoom control prizm data list

нет кнопки для перехода на СБ navigation menu navigation

Dev ZM, [26 Apr 2024 at 11:30:12]:
то что у некоторых нет это так и должно быть

пока нет технической возможности покрыть все примеры

пришлось оставить возможность только v4 для открытия примера, потому что наши примеры после иконок изменились

запушил

@ZurabDev ZurabDev added the availability:reviewed Ready to test label May 6, 2024
@alexhawkins94
Copy link

Listing item with buttons and instruments - ошибка
chrome_69FLcR9AwS

@alexhawkins94
Copy link

Counter directive пример, ошибка
chrome_BWWoymfYkA

@alexhawkins94
Copy link

counter base пример, ошибка
chrome_gWhI8pxzd8

@alexhawkins94
Copy link

Listing item, base пример - шрифты отличаются от наших
СБ:
chrome_Tm7jy5DRfj
Витрина:
chrome_GJKmcOVaIl

@alexhawkins94
Copy link

alexhawkins94 commented May 6, 2024

Confirm dialog, custom button пример - поправить стили, шрифт.
аналогично для : Confirm Dialog Footer Template

chrome_BSZwKm8S0l
у нас:
chrome_3BB5e9br3j

плюс по возможности исправить опечатку на кнопке "подтвердить"

@alexhawkins94
Copy link

Dialog With Parent - ошибка
Dialog Outer Header With Context - ошибка
Dialog Base - ошибка

'prizm-doc-documentation' is not a known element

chrome_uxi4B0q5Kp

@alexhawkins94
Copy link

Поправить шрифты в диалоге и сайдбаре
image

@alexhawkins94
Copy link

Radio Button Small Size - в примере на СБ радиобаттон не small size (16х16), а обычного размера
Аналогично пример big size

@ZurabDev ZurabDev added availability:reviewed Ready to test and removed state:reopen labels May 16, 2024
@alexhawkins94
Copy link

Dialog Outer Header With Context - ошибка при открытии примера в SB
chrome_nb1ZL823D9

@alexhawkins94
Copy link

в рамках этого РМ есть еще что доделать, есть еще что дальше проверять в других компонентах - предлагаю декомпозировать.
в текущий ближайших rc включить этот мр, в котором часть проблем исправлена.
в следующем релизе доделать все остальное

@ZurabDev
@ickisIckis

@ZurabDev
Copy link
Contributor Author

в рамках этого РМ есть еще что доделать, есть еще что дальше проверять в других компонентах - предлагаю декомпозировать. в текущий ближайших rc включить этот мр, в котором часть проблем исправлена. в следующем релизе доделать все остальное

@ZurabDev @ickisIckis

Эту часть включить в релиз, дальше будем декомпоизрвоать, так как задача может стать очень большой

@ZurabDev ZurabDev merged commit 47f7d08 into main May 16, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment