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

Fix Profile (split snippets and settings, issue #258) #271

Closed
wants to merge 54 commits into from

Conversation

M4XPRD
Copy link
Contributor

@M4XPRD M4XPRD commented May 23, 2023

На странице профиля сделал рефакторинг:

  • При заходе в Профиль сразу и для удобства пользователя открываются окна Профиль + Сниппеты (маршрут /profile/snippets/)
  • При клике на Настройки открываются Профиль + Настройки профиля (маршрут /profile/settings)
  • Если в адресной строке написать /profile, то откроется страница с одной колонкой Профиль
  • Добавлен CSS, чтобы колонки выглядели единообразно и ровно
  • Текст выровнен по горизонтали

Деплой: https://maxprd-runit-2.onrender.com

@M4XPRD M4XPRD changed the title Разделить страницу сниппетов и профиля #258 Fix Profile (split snippets and settings, issue #258) May 23, 2023
@ssssank
Copy link
Contributor

ssssank commented May 23, 2023

Здорово! Задеплой, пожалуйста, чтобы можно было посмотреть, как это всё выглядит на деле

@M4XPRD
Copy link
Contributor Author

M4XPRD commented May 23, 2023

Здорово! Задеплой, пожалуйста, чтобы можно было посмотреть, как это всё выглядит на деле

Сделано!

@ssssank
Copy link
Contributor

ssssank commented May 24, 2023

Кажется, чуть логичнее было бы назвать раздел "Профиль", а не "Настройки", а кнопку "Аккаунт" поменять на "Настройки"

@M4XPRD
Copy link
Contributor Author

M4XPRD commented May 24, 2023

Изменил названия в тэгах, поправил стили

@ssssank
Copy link
Contributor

ssssank commented May 24, 2023

@fey возможно криминал, ничего не получается, нужен созвон =D

@M4XPRD M4XPRD closed this May 24, 2023
@M4XPRD M4XPRD deleted the fix-profile branch May 26, 2023 18:40
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.

None yet

4 participants