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

Enhancement 338 sidebar menu modal #344

Merged
merged 3 commits into from
May 6, 2024

Conversation

AlexanderMorugin
Copy link
Collaborator

  • Модальное окно SideBarMenuModal лежит в папке features.

  • Посмотреть модалку можно в сторибуке, либо на странице сайта, через npm run start кликнув в шапке сайта "Помощь" -> "Возвраты". Сжать страницу меньше 768px в ширину и нажать на кнопку "Меню", которая будет расположена над формой.

  • Кнопка "Меню" для перехода в модалку лежит в папке entities -> SideBarButton. Она есть в сторибуке. А на странице сайта "Помощь" -> "Возвраты", кнопка появляется при разрешения экрана менее 768px в ширину.

Copy link
Collaborator

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

Отличная работа!


import styles from './SideBarSublinks.module.scss'

export interface IData {
Copy link
Collaborator

Choose a reason for hiding this comment

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

в компоненте храним только типизацию пропсов, остальное выносим в папку model

@@ -0,0 +1,54 @@
import { Routes } from '@/shared/config/routerConfig/routes'

// import { IData } from '../types/types'
Copy link
Collaborator

Choose a reason for hiding this comment

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

удаляем комментарии, если не нужны

@AlexanderMorugin AlexanderMorugin merged commit 5e59747 into master May 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request верстка
Projects
None yet
2 participants