-
Notifications
You must be signed in to change notification settings - Fork 123
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
Closed #434 #437
Closed #434 #437
Conversation
@bobronaud вижу косяки на лендосе, что нужн опоправить |
@fey да, я тоже видел косяки в лэнде и еще другие, просто подумал что не стоит их исправлять в рамках этого ПР, чтобы каши не было, а сделать отдельный ПР для всех фиксов |
ну смотрите, тут два варианта
|
окей, внесу фиксы в этот пр в течении пары дней |
@fey форму в футере мне закоментить, потому что фичи пока нет, или полностью удалить? |
у нас отправка писем щас не работает. И..зачем она нам?) |
Можете удалить целиком. |
да, это пока норм. футер скорее всего обновим, а страницу about сделаем отдельно |
@fey всё, готово. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
визуально ок
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
по коду тоже всё ок
визуально меня смущает:
|
Надо какой-то один, может бутстраповский. Потом можем перейти на тот, что в самом Хекслете испольузется. @bobronaud можете лого поправить? По поводу шрифтов которые только на ленде -думаю пока некритично, а лого стоит поправить. |
@fey я тестирую переключение темы, у меня с лого нет проблем, переключается цвет как надо. Как я могу воспроизвести проблему с лого? |
нашёл проблему, ща сделаю |
лого пофиксил @fey |
Deploy: https://bobronaud-runit.onrender.com