Photos are in low resolution
Bug #1381086 reported by
Riccardo Padovani
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Telegram app |
Fix Released
|
Medium
|
Roberto Mier Escandon | ||
Bug Description
when you click on photo this is zoomed - and that's awesome - but the photo is in low resolution, like it's the preview that is zoomed. I expect that the full photo is downloaded and show when I click on it
Related branches
lp:~libqtelegram-team/telegram-app/high-res-photos
- Michał Karnicki (community): Approve
-
Diff: 1687 lines (+671/-191)19 files modifiedlib/types/filelocation.h (+15/-9)
qmlplugin/data.cpp (+450/-165)
qmlplugin/data.h (+25/-3)
qmlplugin/dbmanager.cpp (+42/-2)
qmlplugin/dbmanager.h (+2/-0)
qmlplugin/dbschema.h (+39/-8)
qmlplugin/models/contactitem.cpp (+2/-0)
qmlplugin/models/contactitem.h (+5/-0)
qmlplugin/models/dialogsmodel.cpp (+23/-2)
qmlplugin/models/dialogsmodel.h (+2/-0)
qmlplugin/models/dialogsview.cpp (+1/-0)
qmlplugin/models/groupmodel.cpp (+17/-2)
qmlplugin/models/groupmodel.h (+5/-0)
qmlplugin/models/messagesmodel.cpp (+13/-0)
qmlplugin/models/messagesmodel.h (+2/-0)
qmlplugin/telegramclient.cpp (+10/-0)
qmlplugin/telegramclient.h (+4/-0)
qmlplugin/telegramservice.cpp (+10/-0)
qmlplugin/telegramservice.h (+4/-0)
lp:~libqtelegram-team/telegram-app/app-high-res-photos
- Michał Karnicki (community): Needs Fixing
-
Diff: 310 lines (+71/-64)5 files modifiedbootstrap.sh (+0/-55)
scope/data/com.ubuntu.telegram_sctelegram.ini (+3/-0)
ui/GroupProfilePage.qml (+25/-6)
ui/PreviewPage.qml (+17/-0)
ui/ProfilePage.qml (+26/-3)
| Changed in libqtelegram: | |
| importance: | Undecided → Medium |
| Changed in libqtelegram: | |
| status: | New → Confirmed |
| Changed in libqtelegram: | |
| assignee: | nobody → Roberto Mier Escandón (rmescandon) |
| status: | Confirmed → In Progress |
| Changed in libqtelegram: | |
| milestone: | none → m17 |
| Changed in libqtelegram: | |
| status: | In Progress → Fix Committed |
| Changed in libqtelegram: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Bit hard to verify on the device, but images appeared reasonably hi-res to me.