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

Bug: длинный урл ломает верстку #191

Open
fey opened this issue Aug 1, 2023 · 15 comments
Open

Bug: длинный урл ломает верстку #191

fey opened this issue Aug 1, 2023 · 15 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@fey
Copy link
Collaborator

fey commented Aug 1, 2023

Summary

таблица не влезает в колонку, если урл очень длинный

image

Reproduction steps

Добавить опечатку с урлом, типа `https://guides.hexlet.io/ru/bootstrap/?_gl=1*uafu67*_ga*MTAwNjg4MjEyNC4xNjg5MzIxNTQw*_ga_PM3R85EKHN*MTY5MDU0NjAzOC4xMTQuMS4xNjkwNTQ2MDk4LjAuMC4w*_ga_WWGZ6EVHEY*MTY5MDU0NjAzOC4xMTMuMC4xNjkwNTQ2MDM4LjYwLjAuMA..` (и более)

Expected result

Верстка выглядит корректно, в две колонки

Actual result

На скриншоте.

Browsers

Chrome

OS

Linux

@fey fey added the bug Something isn't working label Aug 1, 2023
@fey
Copy link
Collaborator Author

fey commented Aug 16, 2023

Можно ограничить отображаемую ссыылку url. Либо посмотреть, можно ли сделать перенос текста с помощью css

@bazilval
Copy link
Collaborator

bazilval commented Sep 26, 2023

Может имеет смысл ещё и срезать query params и сохранять нормализованный url? По факту они не имеют смысла в рамках наших задач (виджет будет вешатьтся на всю страничку, а не страничку с query params)

Таким образом ещё можно избежать баги с сохранением одной и той же странички, но с разными query params

@fey
Copy link
Collaborator Author

fey commented Sep 26, 2023

@bazilval параметры могут использоваться на страницах в форме. Представьте что у вас 10к записей (тех же опечаток), и пользователь (например в интернет магазине) прислал опечатку. Если вы просто глянете на страницу, то не найдете в чем дело. Поэтому важно нам иметь доступ к странице и открывать ее.

Скорее всего нужно делать обрезку обрезку ссылки или поправить верстку. Но ссылку оставлять оригинальную. Плюсом думаю должна быть кнопка типа "перейти по ссылке опечатки"

@bazilval
Copy link
Collaborator

@fey мне казалось, что виджет нам отдаёт фактическую ссылку, когда делает пост-запрос для публикации репорта. Оттуда мы и можем взять актуальную для конкретной опечатки ссылку и переходить по ней.

А здесь речь об общей ссылке, характеризующей страницу, куда добавлен виджет.

@fey
Copy link
Collaborator Author

fey commented Sep 26, 2023

Не совсем понял. По моему там отображается урл, с которого отправили опечатку.

@bazilval
Copy link
Collaborator

@fey а, только теперь увидел, что речь именно про ссылку в опечатке

я просто смотрел на ссылку, которая в воркспейсе и там тоже вёрстка едет некрасиво, когда длинная ссылка
image

@bazilval
Copy link
Collaborator

@fey тогда думаю можно сделать так, что в таблице с опечатками url отображаются сокращёнными с многоточием на конце и при этом являются активными ссылками, чтобы можно было сразу перейти

возможно даже вообще имеет смысл скрыть ссылку, чтобы просто была активная ссылка "Ссылка"

@fey
Copy link
Collaborator Author

fey commented Sep 26, 2023

Прятать пока не нужна, а обрезать - скорее всего. Я бы глянул в сторону верстки наверное, кажется что через нее должно решаться

@bazilval
Copy link
Collaborator

@fey Ну я через вёрстку и предлагаю, просто ссылке можно навесить класс Bootstrap "text-truncate" и style="max-width:450px;", получится вот так:
image

@fey
Copy link
Collaborator Author

fey commented Sep 27, 2023

@bazilval ок, давайте. Только почему 450px?
Еще, я бы показал ПР (когда появится) верстальщикам Хекслета, чтобы посмотрели и дали совет как лучше сделать.

@bazilval
Copy link
Collaborator

Ну вот с этим надо ещё решить, да
Пусть верстальщики скажут как лучше
Я отталкивался от того, что при масштабе 100% это максимальная ширина, при которой не переносятся остальные колонки

@fey
Copy link
Collaborator Author

fey commented Sep 27, 2023

Я отталкивался от того, что при масштабе 100% это максимальная ширина, при которой не переносятся остальные колонки

да, но это же от ширины экрана зависит?) У меня например есть второй монитор, он почти квадратный)

@bazilval
Copy link
Collaborator

Резонно)
Ну тут уж надо просто чтобы люди более разборчивые посмотрели

@bazilval
Copy link
Collaborator

@fey назначишь на меня?

@fey
Copy link
Collaborator Author

fey commented Mar 11, 2024

Актуально.

@fey fey added the good first issue Good for newcomers label Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants