*** Dima2387 is now known as Guest4405 | 06:35 | |
poetaster | attah, so now on to orientation ;) | 08:51 |
---|---|---|
attah | I think that is a Qt limitation in general | 08:54 |
attah | They forgot to read Exif data way back when, and insist on maintaining bug compatibility | 08:55 |
poetaster | hmmm. it's a really simple fix. setAutoTransform(true) with QImageReader. done. | 09:16 |
attah | Indeed, but i can only assume they are keeping it by choice, e.g. in QImage itself | 09:17 |
attah | and whatever the implicit pipeline you are hitting with pure QML consists of | 09:18 |
poetaster | I found the pipeline, it's a bit convoluted ;) | 09:19 |
poetaster | sigh: I think it's automatic by default in qt 6 | 09:21 |
attah | jaay, finally | 09:23 |
poetaster | Maybe it's actually a 'feature'. to keep c++ programmers employed ;) | 09:24 |
flypig | poetaster, could you elaborate on the issue a bit for me please? If you set autoTransform in the QML, does it work as expected? Is there a reason why you can't do that in general? | 09:37 |
poetaster | autoTransform is not a property of Canvas. Image, yes, no problem, | 09:40 |
flypig | Okay, that clarifies, thank you! | 09:47 |
flypig | And thanks for posting the same on your bug report. | 09:47 |
poetaster | thank you! If it helps I could build a stripped down demo of the behaviour. | 09:48 |
flypig | That would actually be *really* useful, if it's not too much effort? | 09:48 |
poetaster | Will do. | 09:49 |
flypig | Thank you! | 09:49 |
poetaster | flypig, https://github.com/poetaster/canvas-orientation | 12:16 |
poetaster | page one, camera jpeg, portrait, with meta, page two, jpeg orientation done on export, no meta. | 12:17 |
flypig | Many thanks poetaster. So they both render the same, but the first should be rotated? | 12:19 |
flypig | Sorry, I mean to say "So they both render the same way..." | 12:23 |
flypig | "... but the first should be rotated, but isn't". | 12:23 |
flypig | poetaster, did you say that the behaviour had changed in Qt6? | 12:30 |
poetaster | I'm not sure, but the qquickimageprovider.h has a requestImage with a requestedAutoTransform in the signature. | 12:33 |
flypig | Do you have a link for that? I'm not seeing it in the docs: https://doc.qt.io/qt-6.2/qquickimageprovider.html | 12:36 |
poetaster | sorry, in qt-dec | 13:21 |
poetaster | https://github.com/sailfishos/qtdeclarative/blob/mer-5.6/src/quick/util/qquickimageprovider.h | 13:21 |
poetaster | flypig, this is qquckcanvasitem.cpp where the loadimage method is implemented: https://github.com/sailfishos/qtdeclarative/blob/003a2cbafe7d5c036fd2e6b32b9aaa1bd447dc86/src/quick/items/context2d/qquickcanvasitem.cpp | 13:59 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!