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

Sticky upvote #1264

Merged
merged 16 commits into from
Nov 12, 2024
Merged

Sticky upvote #1264

merged 16 commits into from
Nov 12, 2024

Conversation

trin4ik
Copy link
Contributor

@trin4ik trin4ik commented Nov 12, 2024

https://vas3k.club/post/26312/#comment-7aecc765-34b4-421c-a529-acf52006a148

по просьбам трудящихся. проверил на pc/ipad разрешениях и на мобилке. хорошо бы проверить на всяких кастомных постах, типа батлов и прочих, но это, кмк, проще уже на проде прогнать.

iPad

PC

IMG_0530.MP4

@trin4ik trin4ik requested a review from vas3k as a code owner November 12, 2024 08:07
@trin4ik
Copy link
Contributor Author

trin4ik commented Nov 12, 2024

image
я как обычно просто смержил мастер в новую ветку )) прастити

@vas3k
Copy link
Owner

vas3k commented Nov 12, 2024

Мне нравится как оно выглядит на десктопе, но не нравится на мобиле. С этими двумя ездящими по экрану стрелочками, закрывающими контент, выглядит говено. Предлагаю на мобильном CSS пока отключить его нафиг, замержить десктоп, а там подумать как сделать лучше в отдельном PR

@trin4ik
Copy link
Contributor Author

trin4ik commented Nov 12, 2024

Мне нравится как оно выглядит на десктопе, но не нравится на мобиле. С этими двумя ездящими по экрану стрелочками, закрывающими контент, выглядит говено. Предлагаю на мобильном CSS пока отключить его нафиг, замержить десктоп, а там подумать как сделать лучше в отдельном PR

под планшеты оставляем? сейчас есть css под 1024+, т.е. под pc и всё, что меньше. могу сделать 768+ и всё что меньше, тогда на планшетах будет как на десктопе, а в мобилке как было. ну или просто оставить стики только для pc.

@vas3k
Copy link
Owner

vas3k commented Nov 12, 2024

под планшеты оставляем?

Надо посмотреть, если оно на планшетах не перекрывает текст, а ездит рядом — то оставляем. А если наезжает — то не надо

@vas3k vas3k merged commit e3d72d7 into vas3k:master Nov 12, 2024
6 checks 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.

2 participants