Изменение отображения опросов

А что с версткой не так?

https://i.ibb.co/gDJChrF/2023-11-21-00-05-03.png

Пингвин сидит у конкретного варианта ответа, это так и надо?

Пингвин сидит у конкретного варианта ответа, это так и надо?

Да, это небольшая «плюшка».

Ааааа, это он сидит у самого популярного варианта ответа. Понял. Но не сразу если честно, раз 5 смотрел на результаты и на главную :D

Пингвином показываются выбранные варианты проголосовавшим. Чтобы он помнил за что голосовал и ему было стыдно.

На десктопе все в целом ok, разве что выделение выбранного элемента в результатах в топике/ленте слабо заметно.

На мобильном экране всё довольно плохо влезает из-за попытки отобразить название варианта и результаты в одной строке. В итоге и текст не удобно читать из-за кучи переносов, и мало строк влезает.

Он сидит у того варианта, за который ты проголосовал.

Ааааа, вон чего, просто совпало значит с самым популярным. Тогда значит я два раза не понял, сначала думал вёрстка поехала, потом подумал что это самый популярный ответ, а это пометка лично моя где я тыкнул.

что выделение выбранного элемента в результатах в топике/ленте слабо заметно.

там «bold» тег банальный, можно сделать по аналогии с бейджем «автор топика»

плохо влезает из-за попытки отобразить название варианта и результаты в одной строке. В итоге и текст не удобно читать из-за кучи переносов, и мало строк влезает.

можно как решение «в лоб» вставить сокращалку текста на css, которая все что длинее лимита заменит на ..

Но, я видимо сегодня очень непонятливый

Общая идея в том, что форма голосования будет показываться в доступных для голосования опросах

Я не распарсил, форма голосования показывается и так в доступных для голосования опросах не? В смысле так и было же. Я вообще не заметил что изменилось если честно, кроме пингвинёнка, всё вроде как и было… Хотя ладно всё работает, а остальное мелочи, это моя личная проблема что мне разницу надо высматривать на картинках «найди 10 отличий»

А с каких пор можно стало голосовать в не-текущем опросе? Только не говорите, что с того дня, как опрос сменили.

можно как решение «в лоб» вставить сокращалку текста на css, которая все что длинее лимита заменит на ..

Будет не читаемо. За основу можно взять отображение результатов опроса в Telegram: оно там сделано в две строки, на первой проценты и текст, а на второй полоска с результатами. Только придется помудрить с CSS чтобы сохранить десктопный вариант. Можно использовать, например, grid layout и медиа запросы.

Я вообще не заметил что изменилось если честно, кроме пингвинёнка, всё вроде как и было…

Это по сути багофикс, так что если ты проблем не видишь значит все хорошо.

Кривое отображение опросов в мобильной версии

Испортился вид кнопки разворачивания свёрнутого длинного блока кода. Хотя наверно это не связано.

Сделай картинку и оправь вот в этот топик: Обновил подсветку синтаксиса (highlight.js).

Ну понятно, надо было сразу сказать что есть проблема, на простых тестах такие косяки не видны )

Давай поправлю тогда, отдельным пул-реквестом.

что выделение выбранного элемента в результатах в топике/ленте слабо заметно.

там «bold» тег банальный, можно сделать по аналогии с бейджем «автор топика»

вот это делаем?

там «bold» тег банальный, можно сделать по аналогии с бейджем «автор топика»

вот это делаем?

Ага.

Голосование без выбора (или параметр results=true) дает посмотреть результаты, так задумано?

Поясни.

Если пользователь ещё не проголосовал он все равно может увидеть текущий результат, нажав кнопку «Проголосовать» не выбрав никакой из вариантов.

Всегда можно было.

Результаты прятать не планировали, но по-умолчанию теперь показывается форма.

По поводу голосования пустой формой я посмотрю что можно сделать, возможно достаточно будет атрибут “required” воткнуть.

maxcom, почему на мобильной вёрстке вместо пингвина голубой прямоугольник?

Это надо бы поправить. alex0x08 ты сейчас что-то делаешь в районе отображения опросов?

Да, сегодня будет пул-реквест с доработками. На мерж больше времени ушло, не заметил сразу что ты реализацию опросов так сильно успел переделать )

Длинный текст в выбранных пунктах не переносится и наезжает на проценты.

https://ibb.co/x3bw9fP

alex0x08, не всё идеально.

https://i.imgur.com/1ecWX9Z.png

Интересно, причем нижний блок спокойно переехал на вторую строчку.

Скорее всего это от того, что ты использовал готовый стиль из другой части сайта и у него стоит запрет на перенос. Это про выделение своего выбора.

Потому что в классе user-tag установлено white-space: nowrap

Блин, жаль, я готовый и использовал чтобы не плодить стили. Вообщем выложу правку вместе с коммитом по новой фиче.

Лучше, конечно, фиксы отдельными pull request’ами передавать.

так там один файл и пара строчек, больше геммороя этот пул-реквест сделать чем поправить.

ok

Лучше плодить стили, тут есть пользователи, правящие отображение сайте Stylus'ом или ещё чем.

Вёрстка кривая. Мой ответ с btrfs подъехал вплотную к счётчику голосов. Зачем выкатывать сырое? https://ibb.co/fq03RWj

alex0x08

Пингвины классные, но уж больно мыльные. Может вектор туда? Или просто разрешение повыше.

Мой ответ с btrfs подъехал вплотную к счётчику голосов.

Ужос какой, теперь спать ночами не буду прям (нет).

Зачем выкатывать сырое?

А ты разумеется готов помогать с тестированием да?

Причём тут спать или не спать. Вёрстка кривая, это факт.

Чтобы тестировать, нужно знать, что нужно что-то тестировать. Либатсрал нот фоунд. Или мне нужно каждый деть lorsource мониторить?

Выкатили апдейт, напиши если еще где-то вылезут проблемы с версткой.

Теперь выглядит хорошо. Спасибо.

На мобилке пингвин выглядит очень пиксельным, может быть его в svg запихнуть?

скриншот?

https://dropmefiles.com/mbHmx

Надо разрешение увеличить в два раза.

Нда, проклятый 21век и HDPI на телефонах )

maxcom что скажешь? переводить пингвина в вектор?

Я вот думаю может убрать там жирный шрифт и уменьшение размера? Или не нужно?

Достаточно просто положить картинку в 2х или 3х разрешении.

А она есть в ресурсах в большом разрешении? Большой пингвин же с боксерскими перчатками. Или из интернета взять?