vortexwolf / 2ch-Browser

Source code of the 2ch Browser Android application
https://play.google.com/store/apps/details?id=com.vortexwolf.chan
MIT License
86 stars 16 forks source link

новый просмотрщик картинок очень плох #45

Closed magicgoose closed 9 years ago

magicgoose commented 10 years ago

есть 2 проблемы первая — он не позволяет увидеть картинку в полном размере (можно растянуть её, но детализации это не добавит — картинка просто будет замыленной) вторая — если по-быстрому пролистать несколько картинок подряд, приложение падает (вангую, что из-за недостатка памяти)

проверял на sony xperia z

если это всё нельзя пофиксить, сделай хотя бы, чтобы можно было в настройках выбрать старый просмотрщик...

vortexwolf commented 10 years ago

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

magicgoose commented 10 years ago

но если открыть картинку ОП из списка тредов, она открывается по-старому, можно увидеть в полном размере, и ничего не вылетает. хардкодить лимит в 800 пикселей — это как-то странно, у девайсов памяти разное количество и экраны разные. на экране с большим dpi такой лимит выглядит несколько странно реально, может просто добавить опцию использования старого просмотрщика? с ним всё заебок было и без уменьшения.

vortexwolf commented 10 years ago

800 правильное значение, т.к. с ним приложение не вылетает, а если поставить 1000, то вылетает гораздо чаще. Все же добавил в опциях чекбокс для возврата предыдущего просмотрщика.

magicgoose commented 10 years ago

ты уж определись, вылетает чаще, или просто вылетает а то получается, что и с 800 вылетает (в принципе, так и есть)

вообще это решается использованием очищаемого кэша (очищать можно например отсюда)

за чекбокс огромное спасибо

vortexwolf commented 10 years ago

Если даже с 800 вылетает - тогда надо менять телефон на менее глючный и менее китайский. И тем более будут проблемы в каком-нибудь gif-треде, где каждая картинка по 4Мб. На данный момент в последней версии 1.4.2 поменял просмотрщик на галерею из браузеров. Теперь проблема 1 исправлена, а проблема 2 останется у кого-то надолго, т.к. у меня не воспроизводится.

magicgoose commented 9 years ago

судя по наличию в коде класса FixedSubsamplingScaleImageView и его описанию, проблемы уже не должно быть? если так и есть, то пора этот issue выпилить 8)

vortexwolf commented 9 years ago

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