Skip to content

Finished makin new Sidebar#713

Merged
Valyukhov merged 23 commits into
hiscoder-com:developfrom
BogdanLi:feature-bogdan-690
Oct 22, 2024
Merged

Finished makin new Sidebar#713
Valyukhov merged 23 commits into
hiscoder-com:developfrom
BogdanLi:feature-bogdan-690

Conversation

@BogdanLi
Copy link
Copy Markdown
Collaborator

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 16, 2024

Deploy Preview for level-bible ready!

Name Link
🔨 Latest commit fade805
🔍 Latest deploy log https://app.netlify.com/sites/level-bible/deploys/6717d78c3309980008d0422d
😎 Deploy Preview https://deploy-preview-713--level-bible.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Valyukhov
Copy link
Copy Markdown
Contributor

ширина экрана 1024-1536 выглядит вот так:
image

@Valyukhov
Copy link
Copy Markdown
Contributor

по дизайну ширина сайдбара - 50px, у тебя намного больше

@Valyukhov
Copy link
Copy Markdown
Contributor

иконки очень сильно отличаются от дизайна, я думал ты такие же найдёшь
image

тогда лучше брать из дизайна в фигме
напиши в чём сложность у тебя с ними

@Valyukhov
Copy link
Copy Markdown
Contributor

image
перевод работает только на странице account

@Valyukhov
Copy link
Copy Markdown
Contributor

кнопка История версий должна быть на всю ширину
image

@Valyukhov
Copy link
Copy Markdown
Contributor

Нет разделителей у среднего блока - темы
image

@Valyukhov
Copy link
Copy Markdown
Contributor

image
содержимое аккордеона выходит за предел линии иконок

@Valyukhov
Copy link
Copy Markdown
Contributor

Valyukhov commented Oct 18, 2024

  1. Сайдбар должен занимать всю высоту минус высоту аппбара(как в дизайне).

  2. Аппбар везде зафиксирован на всех страницах.

  3. Логотип выровнять по левой границе контейнера
    image

  4. В дизайне есть вариант с неактивными пунктами меню. Надо его сделать.
    Когда открыта страница или модалка из этого пункта, то её делать активной
    image

5.Отступы сайдбара в развернутом виде сделаны не по дизайну
6. Размер шрифтов также не соответсвуют
image

@Valyukhov
Copy link
Copy Markdown
Contributor

При нажатии на Блокнот - модалка не открывается

@Valyukhov
Copy link
Copy Markdown
Contributor

Перестали работать модалки

@Valyukhov
Copy link
Copy Markdown
Contributor

Нужно написать Connect with us как в дизайне:

  1. Капслоком.
  2. С отступами.
  3. После успешной отправки надпись Connect with us убираем.
    image

@Valyukhov
Copy link
Copy Markdown
Contributor

image

Поменять название кнопок
История версий/Последняя версия
На английском: Version history / Latest version
На испанском: Historial de versiones / Última versión

@Valyukhov
Copy link
Copy Markdown
Contributor

Valyukhov commented Oct 18, 2024

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

@Valyukhov
Copy link
Copy Markdown
Contributor

Во всех модалках выровнять по центру название. Сейчас оно смещено чуть влево.
image

@Valyukhov
Copy link
Copy Markdown
Contributor

Фон за модалкой в дизайне поменялся - нужно исправить по всему сайту.
Он стал темней и убрали блюр.
image

Comment thread components/AppBar.js Outdated
Comment thread components/ModalInSideBar.js Outdated
Comment thread components/SwitchLocalization.js Outdated
Comment thread components/SwitchLocalization.js Outdated
Comment thread components/ThemeSwitcher.js Outdated
Comment thread components/ThemeSwitcher.js Outdated
Comment thread components/ThemeSwitcher.js Outdated
Comment thread components/SignOut.js Outdated
Comment thread components/SignOut.js Outdated
Comment thread components/SideBar.js
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

очень уж тяжёлый получился компонент, как думаешь можно разбить? или тяжело по логике?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

это не обязательно, я понимаю специфическая логика, часть элементов - ссылки, часть - модалки, часть - другие компоненты.
просто напиши идеи, если будут

Comment thread components/SideBar.js Outdated
Comment thread components/SideBar.js Outdated
Comment thread components/SideBar.js Outdated
Comment thread components/SideBar.js Outdated
Comment thread components/SideBar.js Outdated
Comment thread components/SideBar.js Outdated
Copy link
Copy Markdown
Contributor

@Valyukhov Valyukhov left a comment

Choose a reason for hiding this comment

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

Есть комментарии

Comment thread components/SideBar.js Outdated

const [collapsed, setCollapsed] = useState(true)
const [isLargeScreen, setIsLargeScreen] = useState(false)
const [isOpen, setIsOpen] = useState(false)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

isOpen используется где-то?

@Valyukhov Valyukhov merged commit 46abc9b into hiscoder-com:develop Oct 22, 2024
@BogdanLi BogdanLi deleted the feature-bogdan-690 branch October 25, 2024 02:58
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.

2 participants