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/organizations page #25

Merged
merged 82 commits into from
Jun 19, 2024
Merged

Feat/organizations page #25

merged 82 commits into from
Jun 19, 2024

Conversation

kirill-ivanovvv
Copy link
Member

Closes #23

@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov June 11, 2024 08:42
@kirill-ivanovvv kirill-ivanovvv self-assigned this Jun 11, 2024
@kirill-ivanovvv
Copy link
Member Author

@ssagg
Copy link
Contributor

ssagg commented Jun 11, 2024

@ssagg, смотри, тему разделил на 2 директории - dark & light переключатель темы сделал - работает вот этот фрагмент - зафиксировал

принял

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

так же не прошли чеки

@kirill-ivanovvv kirill-ivanovvv force-pushed the feat/organizations-page branch from d4fb92b to 87e9762 Compare June 12, 2024 11:11
@kirill-ivanovvv
Copy link
Member Author

@ssagg обновил тему

@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov June 14, 2024 16:09
@ssagg
Copy link
Contributor

ssagg commented Jun 14, 2024

дд

@effektivnayarabota1 у нас сейчас модалки по центру открываются, верно? а если надо в произвольном месте ее открыть, например, как notifications в макете ? Screenshot from

нужно так делать: на уровне фрагмента чтоли дописать такой функционал. я темную теме на странице добью, этим вопросом займусь. Ближе к вечеру image

@effektivnayarabota1 я модифицировал компонент модалки. по дефолту все остается, как есть. но можно зафиксировать модалку и позицию настроить. как раз пригодился бокс из нашего компонента Layout. Обратная совмстимость осталась везде.

@kirill-ivanovvv
Copy link
Member Author

дд

@effektivnayarabota1 у нас сейчас модалки по центру открываются, верно? а если надо в произвольном месте ее открыть, например, как notifications в макете ? Screenshot from

нужно так делать: на уровне фрагмента чтоли дописать такой функционал. я темную теме на странице добью, этим вопросом займусь. Ближе к вечеру image

@effektivnayarabota1 я модифицировал компонент модалки. по дефолту все остается, как есть. но можно зафиксировать модалку и позицию настроить. как раз пригодился бокс из нашего компонента Layout. Обратная совмстимость осталась везде.

я вот здесь лейаут для страниц добавл. у меня почему-то если роуты index.ts называешь, они не открываются. по докам page.ts указанно, я так сделал - работает

@ssagg
Copy link
Contributor

ssagg commented Jun 17, 2024

дд

@effektivnayarabota1 у нас сейчас модалки по центру открываются, верно? а если надо в произвольном месте ее открыть, например, как notifications в макете ? Screenshot from

нужно так делать: на уровне фрагмента чтоли дописать такой функционал. я темную теме на странице добью, этим вопросом займусь. Ближе к вечеру image

@effektivnayarabota1 я модифицировал компонент модалки. по дефолту все остается, как есть. но можно зафиксировать модалку и позицию настроить. как раз пригодился бокс из нашего компонента Layout. Обратная совмстимость осталась везде.

я вот здесь лейаут для страниц добавл. у меня почему-то если роуты index.ts называешь, они не открываются. по докам page.ts указанно, я так сделал - работает

@effektivnayarabota1 да, если app роутинг использовать, то надо page все называть. Переименовал свои страницы тоже. Предлагаю папку management переименовать в dashboard, это же в пути в браузере будет. В крайнем случае скобками потом изолируем. Посмотри мой послдений коммит можешь скачать. там по дефолту идет редирект на dashboard/users и по сслыке registration вторая страница. потом напишем мидлвару авторизациии и protected route. еще надо решить по стилям потом. в лэйауте их оставить или во фрагментах.

@kirill-ivanovvv
Copy link
Member Author

дд

@effektivnayarabota1 у нас сейчас модалки по центру открываются, верно? а если надо в произвольном месте ее открыть, например, как notifications в макете ? Screenshot from

нужно так делать: на уровне фрагмента чтоли дописать такой функционал. я темную теме на странице добью, этим вопросом займусь. Ближе к вечеру image

@effektivnayarabota1 я модифицировал компонент модалки. по дефолту все остается, как есть. но можно зафиксировать модалку и позицию настроить. как раз пригодился бокс из нашего компонента Layout. Обратная совмстимость осталась везде.

я вот здесь лейаут для страниц добавл. у меня почему-то если роуты index.ts называешь, они не открываются. по докам page.ts указанно, я так сделал - работает

@effektivnayarabota1 да, если app роутинг использовать, то надо page все называть. Переименовал свои страницы тоже. Предлагаю папку management переименовать в dashboard, это же в пути в браузере будет. В крайнем случае скобками потом изолируем. Посмотри мой послдений коммит можешь скачать. там по дефолту идет редирект на dashboard/users и по сслыке registration вторая страница. потом напишем мидлвару авторизациии и protected route. еще надо решить по стилям потом. в лэйауте их оставить или во фрагментах.

dashboard - ok
redirect - ok

на счет стилей не понял. ты про theme provider?
на счет изоляции скобками тоже не понял

@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov June 17, 2024 11:08
@ssagg
Copy link
Contributor

ssagg commented Jun 17, 2024

дд

@effektivnayarabota1 у нас сейчас модалки по центру открываются, верно? а если надо в произвольном месте ее открыть, например, как notifications в макете ? Screenshot from

нужно так делать: на уровне фрагмента чтоли дописать такой функционал. я темную теме на странице добью, этим вопросом займусь. Ближе к вечеру image

@effektivnayarabota1 я модифицировал компонент модалки. по дефолту все остается, как есть. но можно зафиксировать модалку и позицию настроить. как раз пригодился бокс из нашего компонента Layout. Обратная совмстимость осталась везде.

я вот здесь лейаут для страниц добавл. у меня почему-то если роуты index.ts называешь, они не открываются. по докам page.ts указанно, я так сделал - работает

@effektivnayarabota1 да, если app роутинг использовать, то надо page все называть. Переименовал свои страницы тоже. Предлагаю папку management переименовать в dashboard, это же в пути в браузере будет. В крайнем случае скобками потом изолируем. Посмотри мой послдений коммит можешь скачать. там по дефолту идет редирект на dashboard/users и по сслыке registration вторая страница. потом напишем мидлвару авторизациии и protected route. еще надо решить по стилям потом. в лэйауте их оставить или во фрагментах.

dashboard - ok redirect - ok

на счет стилей не понял. ты про theme provider? на счет изоляции скобками тоже не понял

например, есть марджин в бейс лайоут и есть марджин в юсер-тайтл и в тем-селектор. но возможно у тебя во фрагментах исправлено. base-layout незначительно поправил, т.к. элемент неправильно отображались
если папку dashboard перименовать в (dashboard), то у нас пусть в адресной строке будет не dashboard/users, а просто users.

@kirill-ivanovvv
Copy link
Member Author

дд

@effektivnayarabota1 у нас сейчас модалки по центру открываются, верно? а если надо в произвольном месте ее открыть, например, как notifications в макете ? Screenshot from

нужно так делать: на уровне фрагмента чтоли дописать такой функционал. я темную теме на странице добью, этим вопросом займусь. Ближе к вечеру image

@effektivnayarabota1 я модифицировал компонент модалки. по дефолту все остается, как есть. но можно зафиксировать модалку и позицию настроить. как раз пригодился бокс из нашего компонента Layout. Обратная совмстимость осталась везде.

я вот здесь лейаут для страниц добавл. у меня почему-то если роуты index.ts называешь, они не открываются. по докам page.ts указанно, я так сделал - работает

@effektivnayarabota1 да, если app роутинг использовать, то надо page все называть. Переименовал свои страницы тоже. Предлагаю папку management переименовать в dashboard, это же в пути в браузере будет. В крайнем случае скобками потом изолируем. Посмотри мой послдений коммит можешь скачать. там по дефолту идет редирект на dashboard/users и по сслыке registration вторая страница. потом напишем мидлвару авторизациии и protected route. еще надо решить по стилям потом. в лэйауте их оставить или во фрагментах.

dashboard - ok redirect - ok
на счет стилей не понял. ты про theme provider? на счет изоляции скобками тоже не понял

например, есть марджин в бейс лайоут и есть марджин в юсер-тайтл и в тем-селектор. но возможно у тебя во фрагментах исправлено. base-layout незначительно поправил, т.к. элемент неправильно отображались если папку dashboard перименовать в (dashboard), то у нас пусть в адресной строке будет не dashboard/users, а просто users.

у меня наржинов нет. есть паддинг общий на всю обертку + gap.
у theme selector абсолютное позиционирование
скриншот приложил.
image

на счет круглых скобок - давай так сделаем, логично все.

@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov June 18, 2024 08:31
@Nelfimov Nelfimov merged commit 3ff84f1 into master Jun 19, 2024
3 checks passed
@Nelfimov Nelfimov deleted the feat/organizations-page branch February 17, 2025 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

organizations page
3 participants