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

NewStory: после прилёта картинки с предпросмотром, журнал не докручивается до конца #4342

Closed
dartraiden opened this issue Apr 8, 2024 · 8 comments

Comments

@dartraiden
Copy link
Member

photo_2024-04-08_22-18-28

@dartraiden dartraiden added the bug label Apr 8, 2024
@dartraiden
Copy link
Member Author

dartraiden commented Apr 8, 2024

На скриншоте: журнал был прокручен до конца. Прилетела картинка с предпросмотром (в 22:15). NS показывает лишь начало картинки, а до конца журнал не докрутил.

@georgehazan
Copy link
Member

Тут ситуация очень хреновая с этим... получается так: событие добавляется телегой в тот момент, когда файла физически еще нет на диске, поэтому оно рендерится на 35 пикселей или две строчки. Потом файл докачивается, выдается событие редактирования события, оно пересчитывает высоту с учетом картинки, но позиционирование по высоте остается тем же

@georgehazan
Copy link
Member

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

@dartraiden
Copy link
Member Author

Однозначно что-то делать надо, т.к. картинку просто не видно, а докручивать каждый раз вручную это маразм.

@SM-Sergey
Copy link

А если перейти на штатные протокольные превьюшки? Их размер заранее известен. И хранить их можно где-то в отдельном кеше, а не грузить в загрузки. Или это очень сложно?

@dartraiden
Copy link
Member Author

dartraiden commented Apr 9, 2024

Не все протоколы имеют штатные. Например, Jabber, где файл просто закачан на сервер по HTTP Upload и прилетает ссылка, которая преобразуется в облачный файл, скачивается и делается превьюшка.

@SM-Sergey
Copy link

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

@SM-Sergey
Copy link

И для "остальных" можно зафиксировать высоту каким-то строго фиксированным размером, а ширину делать - ну какая выйдет по факту.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants