| Summary: | BQM Resize tool : make options more flexible [patch] | ||
|---|---|---|---|
| Product: | [Applications] digikam | Reporter: | Droebbel Melta <droebbel.melta> |
| Component: | Plugin-Bqm-Resize | Assignee: | Digikam Developers <digikam-bugs-null> |
| Status: | REPORTED --- | ||
| Severity: | wishlist | CC: | caulier.gilles, kallewirsch, kde, metzpinguin, sambhavdusad24 |
| Priority: | NOR | ||
| Version: | 1.3.0 | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
Preserve Aspect Ratio by width/height added with some minor changes.
Preserve Aspect Ratio does not work properly. Added height input. Now, image can be resized from both the sides. Minor modifications in preserve aspect ratio by width/height New Improvements Added preserve aspect ratio by longest option. Resets the sliders when unchecked |
||
|
Description
Droebbel Melta
2010-06-14 14:07:46 UTC
Right, there should be more options, like in the resize tool in the image editor. I agree. As others tools shared between editor and BQM, a settings widget need to be writted and used in both. A filter implementation based on mutithreaded filter must be implemented. all this code must be placed in dimg/filters/... Gilles Caulier *** Bug 320362 has been marked as a duplicate of this bug. *** *** Bug 320360 has been marked as a duplicate of this bug. *** *** Bug 320363 has been marked as a duplicate of this bug. *** *** Bug 341757 has been marked as a duplicate of this bug. *** *** Bug 252337 has been marked as a duplicate of this bug. *** I agree with everyone else's assertions on being able to set a desired height and width. I want to add that: I believe there should be an option to resize based on height alone where the width simply scales with it. The ability to use the 'Restore Photograph' option would be great too. I tried reproducing the issue and I was able to resize the image with both the sides. Also, the image can be resized upto 999% of original image which was greater than 4000px. When we right click on the bar, it turns into an input box, so we don't need to implement that. There can be other fetures like some predefined set of dimensions and some user configurable dimensions which they use frequently. Also, having more units like cm or inches would be great for the user. Created attachment 118098 [details]
Preserve Aspect Ratio by width/height added with some minor changes.
Preserve Ratio Checkbox not working properly.
Also having to set a height is not a good idea. Try to reduce images with different resolution and orientation to a common maximum edge length of 1500 pixels. Depending on the orientation, either the width or the height should be 1500 pixels each. That is no longer possible. The BQM can have images with different resolutions whose height and width we do not know. Maik Created attachment 118121 [details]
Preserve Aspect Ratio does not work properly.
Created attachment 118169 [details]
Added height input. Now, image can be resized from both the sides.
Note: Sliders are enabled even when Use Custom is not checked.
Sorry for the inconvenience.
Created attachment 118233 [details]
Minor modifications in preserve aspect ratio by width/height
Your patch can not be applied, you are not in sync with the git archive. Best you load your resize.cpp completely high. Maik I have reset the development/dplugins branch before the first change to the resize tool. Save your resize.cpp and resize.h and syncronize your branch and play back your modified data and create a new patch. Maik Created attachment 118234 [details]
New Improvements
Created attachment 118250 [details]
Added preserve aspect ratio by longest option.
There are still some problems here. In custom mode, the aspect ratio check boxes are grayed out, but the last checked state is retained and processed. I also think that the wish from the description can not yet be realized. Maik Created attachment 118335 [details]
Resets the sliders when unchecked
Thanks for pointing it out.
|