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

Fixed adaptability on the solutions page. Issues #1584 #1652

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

artengin
Copy link
Contributor

Исправил адаптивное отображение фильтра "Пользователь; Упражнение" на странице /solutions

Примеры на экранах шириной больше 768px
Было:
old1
Стало:
new1

Примеры на экранах шириной больше 576px
Было:
old2
Стало:
new2

Примеры на экранах шириной меньше 576px
Было:
old3
Стало:
new3

@fey
Copy link
Collaborator

fey commented Jan 10, 2025

@artengin можете задеплоить демо на render чтобы посмотреть визуально?

@artengin
Copy link
Contributor Author

Ссылка на демо: https://artengin-sicp.onrender.com/

@fey
Copy link
Collaborator

fey commented Jan 15, 2025

@amshkv посмотри, пожалуйста, ок ли ок.

@@ -36,6 +36,19 @@
}
}

@media (max-width: 768px) {
Copy link

Choose a reason for hiding this comment

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

https://getbootstrap.com/docs/5.3/layout/breakpoints/#min-width

а почему не пользуешься бутстраповскими переменными?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Добавил бутстраповские переменные, для этого пришлось создать новый scss файл и подключать его после импорта бутстрап.
Не знаю правильно ли я сделал.
Все работает, но Codacy ругается, потому что не поймет что такое @include

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.

3 participants