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 schools page adaptive colums for small screen #170

Merged
merged 1 commit into from
May 5, 2023

Conversation

Space8rain
Copy link
Contributor

No description provided.

@emp7yhead
Copy link
Contributor

@ssssank чекни пожалуйста

@ssssank
Copy link
Contributor

ssssank commented May 2, 2023

@Space8rain я вижу, что изменений совсем немного, но давай для опыта потренируемся. Нужно задеплоить на railway или render этот код, чтобы можно было "потыкать" и проверить, что всё работает

@Space8rain
Copy link
Contributor Author

@ssssank, ок) клево, что предложил, я даже не знал о таком, пойду поузнаю/почитаю, что это и если не пойму вернусь

@Space8rain
Copy link
Contributor Author

@ssssank, ёу) кажется получилось, но я пока разбирался почему на винде из-за конфликтов в неймингах не переключалась ветка на main, кажется что-то намудрил в коммитах
https://hexlet-comparator-production-1ca3.up.railway.app/schools

@ssssank
Copy link
Contributor

ssssank commented May 3, 2023

Здорово, что разобрался и задеплоил! :)
Остался один вопрос про изменение текста описания

@Space8rain
Copy link
Contributor Author

Ага, вот я про это и говорю, что наворотил ненужного видимо и не понимаю как откатить

@ssssank
Copy link
Contributor

ssssank commented May 4, 2023

Самый простой вариант - просто скопировать нужное обратно и закоммитить

@fey
Copy link
Collaborator

fey commented May 5, 2023

Кажется тут что-то пошло не так и сломались нужные описания.
@Space8rain ты на Хекслете учишься? Можем помочь разрулить.

@Space8rain
Copy link
Contributor Author

@fey, учился по подписке на Хекслете какое-то время, сейчас нет возможности. Было бы круто если поможете.

@fey
Copy link
Collaborator

fey commented May 5, 2023

@Space8rain сможешь сделать самостоятельно ребейз? У тебя появилось много лишних коммитов, их стоит убрать и оставить только нужный с правкой.

@fey
Copy link
Collaborator

fey commented May 5, 2023

и по сообщению линтера тоже стоит попрваить.

@Space8rain
Copy link
Contributor Author

@fey, почитал про ребейз, мне через интерактивный режим нужно удалить комиты ?

@fey
Copy link
Collaborator

fey commented May 5, 2023

@Space8rain ага, если вдруг что-то пойдет не так, можно будет исправить (если помнишь, как правил колонки)

@Space8rain
Copy link
Contributor Author

@fey, так, ну вроде то, что я через rebase сдвинулся к старому коммиту, поудалял свои косячные и запушил через флаг -f, поудаляло лишние ? но теперь конфликты появились

@fey
Copy link
Collaborator

fey commented May 5, 2023

@Space8rain отметься в чатике - https://t.me/hexletcommunity/12
Предлагаю созвониться =)

@Space8rain
Copy link
Contributor Author

@fey, Ура) кажется получилось, но честно говоря не до конца понял, но опыт получен, дальше обучимся) спасибо за помощь

@fey fey merged commit 5902aa1 into Hexlet:main May 5, 2023
1 check passed
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.

На странице "Школы" поправить верстку карточек
4 participants