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

Cropping an image increases the file size (compression ratio) #30

Closed
myii opened this issue Apr 15, 2019 · 4 comments

Comments

@myii
Copy link

commented Apr 15, 2019

@StefWe

This comment has been minimized.

Copy link
Contributor

commented Apr 15, 2019

I have searched in ubuntu-ui-extras and found that the quality is set to 90. So it should compress the picture.

https://doc-snapshots.qt.io/qt5-5.9/qimage.html#save

bool saved = image.save(m_photo->file().filePath(),
m_photo->fileFormat().toStdString().c_str(), 90);

I don't see the why it is not working.

@StefWe

This comment has been minimized.

Copy link
Contributor

commented Apr 15, 2019

With -1 as quality parameter it use the same compression as the picture has before.

Screenshot_20190415_130608

Can we live with that?

@myii

This comment has been minimized.

Copy link
Author

commented Apr 15, 2019

@StefWe As mentioned in our discussion in the Telegram group, using -1 sounds like the right solution in the circumstances.

@UniversalSuperBox

This comment has been minimized.

Copy link
Member

commented Apr 18, 2019

Fixed by #31

OTA-9 automation moved this from To do to QA Apr 18, 2019

@UniversalSuperBox UniversalSuperBox moved this from QA to Done in OTA-9 May 7, 2019

@UniversalSuperBox UniversalSuperBox moved this from Done to QA in OTA-9 May 7, 2019

@UniversalSuperBox UniversalSuperBox moved this from QA to Done in OTA-9 May 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3 participants
You can’t perform that action at this time.