Bug 143340 - Undo/redo for checking/unchecking KeepRatio in Image Properties Dialog not properly handled
Summary: Undo/redo for checking/unchecking KeepRatio in Image Properties Dialog not pr...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.0.0.3 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
: 154162 (view as bug list)
Depends on:
Blocks: Undo-Redo
  Show dependency treegraph
 
Reported: 2021-07-13 20:19 UTC by Stefan_Lange_KA@T-Online.de
Modified: 2024-05-07 07:21 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments
Test document to show the problem (93.67 KB, application/vnd.oasis.opendocument.text)
2021-07-13 20:19 UTC, Stefan_Lange_KA@T-Online.de
Details
Screenshot with https://gerrit.libreoffice.org/c/core/+/159835/5 in place (for discussion in Gerrit) (23.02 KB, image/png)
2023-11-27 14:15 UTC, Michael Weghorn
Details
Alternative solution (52.39 KB, image/png)
2023-11-29 16:15 UTC, Heiko Tietze
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan_Lange_KA@T-Online.de 2021-07-13 20:19:54 UTC
Created attachment 173542 [details]
Test document to show the problem

Undo and Redo (indicator arrows as well as actions) are not properly handled for checking and unchecking KeepRatio in the Image Properties dialog.

The problem was found on tests to Bug 143321.

Reproducing the problem:
- open attached document "Test_Bildgröße_V2_keep_ratio_off.odt"
- select the image and open the properties dialog -> KeepRatio is unchecked

- check KeepRatio + exit by OK 
- Result: the Undo arrow is shown blue, Redo arrow is grayed and a red point is added to the Save icon -> OK
- click on the Undo arrow
- Result: the Undo arrow is grayed, Redo arrow is shown blue and a the red point on the Save icon disappeared -> looks ОК
- but: select the image and open the properties dialog again-> KeepRatio is still checked -> not OK!

- uncheck KeepRatio -> Height value is set to Width value (10,00 cm) - problem as described in Bug 143321
- reset Height value manually to 7.5 cm + exit by OK
- Result: Undo arrow, Redo arrow and Save icon are not changed (gray, blue, without red point) -> ОК (???)

- click on the Redo arrow
- Result: Undo arrow is shown blue, Redo arrow is grayed and a red point is added to the Save icon -> looks OK
- but: select the image and open the properties dialog -> looks like before, KeepRatio is unchecked -> not OK
Comment 1 Telesto 2021-07-14 07:49:19 UTC
Confirm
Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 4e0a7df2dfa10bc52d5dbda34f43e0bc0df22ae7
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL
Comment 2 Telesto 2021-07-14 07:56:48 UTC
Also in
6.2

and in
Versie: 4.4.7.2 
Build ID: f3153a8b245191196a4b6b9abd1d0da16eead600
Locale: nl_NL


also in
Versie: 4.1.0.4 
Build ID: 89ea49ddacd9aa532507cbf852f2bb22b1ace28

and in
Versie 4.0.0.3 (Bouw-id: 7545bee9c2a0782548772a21bc84a9dcc583b89)

no undo in
3.5.7.2


Adding bibisectrequest for the implementation of undo. Suspecting an implementationError
Comment 3 Stefan_Lange_KA@T-Online.de 2023-05-15 13:44:16 UTC
After the last change (buzea.bogdan@libreoffice.org, 2023-05-12 15:57:40 UTC) I have tested with current and coming soon versions of LibreOffice if the bug still exists:
Version: 7.5.3.2 (X86_64) / LibreOffice Community
Build ID: 9f56dff12ba03b9acd7730a5a481eea045e468f3
CPU threads: 4; OS: Windows 10.0 Build 19045; UI render: default; VCL: win
Locale: de-DE (de_DE); UI: de-DE
Calc: CL threaded
and
Version: 7.6.0.0.alpha1+ (X86_64) / LibreOffice Community
Build ID: 6f1534940ac12ff8e46f4782e18cfb6cf585da39
CPU threads: 4; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: de-DE
Calc: CL threaded

I have tested as decribed in the bug description (open "Test_Bildgröße_V2_keep_ratio_off.odt" and check "KeepRatio" in the picture properties dialog).
Result with both LO versions:
The behavior has changed completely:
- the Undo arrow remains unchanged (gray)
- the Save icon remains unchanged (no red point is added)
- When I click on the Save icon (without the red point!) the changed document is saved correctly: New entry "KeepRatio" is added to ods subdocument "settings.xml" and at the next open of the document "KeepRatio" appears checked.
- But when I try to close the document without save no question appears "Document changed, save?" and the document is closed without to save the change.

Because of the changed behavior I am not sure if the bug shall remain opened (status "New") or if it should be closed (what reason?) and a new bug should be reported.
Comment 4 Stefan_Lange_KA@T-Online.de 2023-05-15 21:06:47 UTC
I have bibisected to find the change from "old" behavior as described in the bug description to the "new" behavior as described in Comment 3:
 e18b47dbd75cec720d7f486a44b8a235625a5856 is the first bad commit
commit e18b47dbd75cec720d7f486a44b8a235625a5856
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Mar 17 11:22:13 2022 -0700

    source 9e8712ed6f9fb5dbd971e352a5709bd45fadc74f

    source 9e8712ed6f9fb5dbd971e352a5709bd45fadc74f

 instdir/program/cuilo.dll   | Bin 4620800 -> 4621312 bytes
 instdir/program/setup.ini   |   2 +-
 instdir/program/swlo.dll    | Bin 17019392 -> 17019904 bytes
 instdir/program/version.ini |   2 +-
 4 files changed, 2 insertions(+), 2 deletions(-)

The "old" behavior was in the bisect tests exactly as described in the bug description except the detail "Height value is set to Width value (10,00 cm) - problem as described in Bug 143321". This effect propably didn't occur because Bug 143633 was already fixed in LO 7.2.1 on 2021-08-25 15:10:09 UTC.

IMHO it makes no sense to bibisect for the start of the "old" behavior because it is obsolete.
Comment 5 Kira Tubo 2023-11-05 06:36:13 UTC
*** Bug 154162 has been marked as a duplicate of this bug. ***
Comment 6 Buovjaga 2023-11-06 06:48:59 UTC
Let's ask Miklos what he thinks
Comment 7 Miklos Vajna 2023-11-06 08:36:01 UTC
The UI is a bit messy here, I fear. On one hand, "keep ratio" can be a special value for width or height in case the other axis has a fixed value.

On the other hand, this "keep ratio" can be a view option, so if you enabled it for one image, there is some effort to enable it for an other image as well, if you adjust the width or height.

You see how "keep ratio" can mean two different things. :-) One is a UI-level option, the other is something that is stored in the document model for an image.

The linked commit mentions the "80% shows up as 237%" problem, we certainly don't want to return to the old state. But in case somebody wants to improve the UI here so it's less confusing, that would make sense.
Comment 8 Buovjaga 2023-11-06 09:20:53 UTC
(In reply to Miklos Vajna from comment #7)
> The UI is a bit messy here, I fear. On one hand, "keep ratio" can be a
> special value for width or height in case the other axis has a fixed value.
> 
> On the other hand, this "keep ratio" can be a view option, so if you enabled
> it for one image, there is some effort to enable it for an other image as
> well, if you adjust the width or height.
> 
> You see how "keep ratio" can mean two different things. :-) One is a
> UI-level option, the other is something that is stored in the document model
> for an image.
> 
> The linked commit mentions the "80% shows up as 237%" problem, we certainly
> don't want to return to the old state. But in case somebody wants to improve
> the UI here so it's less confusing, that would make sense.

UX team: could that "somebody" be you?!
Comment 9 Stefan_Lange_KA@T-Online.de 2023-11-06 23:33:39 UTC
When in Comment_7 is written "The UI is a bit messy here ..." I think ths is true.

But I am not sure if the information in Comment_7 is completely true.

I have seen the same (or nearly the same) properties dialog is used for different object types (images, frames, ???) but especially the "Keep ratio" setting is handled different.

As far I have seen until now and just tested newly the setting "Keep ratio" from the Image properties dialog is used for all images (!) of a document, means one sets this value once on or off and this setting is used for all images. Furthermore this setting is saved in the odt subdocument "Settings.xml" as config item, e.g. <config:config-item config:name="KeepRatio" config:type="boolean">true</config:config-item>.
So the value is kept when the document is closed and opened again. 

On the other hand the properties dialog e.g. for frames looking nearly - or completely (?) - like the properties dialog for images also contains the setting "Keep ratio" but when it is set on it is valid only for this one call of the dialog. It is lost at the next dialog call for this frame or other frames and it is also not used for images.
Vice versa the "Keep ratio" value set for images is not used for frames.
Comment 10 Miklos Vajna 2023-11-07 07:43:09 UTC
SwViewOption::IsKeepRatio() is what I had in mind for the view-level option (this is similar to e.g. "show non-printable characters"). And then we also have SwFormatFrameSize::QueryValue() for the document model, where a magic 255% percent for the width or height means "just keep the ratio according to the other axis".

Given that we have a single "keep ratio" on the UI, this is confusing, I would say.
Comment 11 Heiko Tietze 2023-11-08 11:16:12 UTC Comment hidden (off-topic)
Comment 12 Heiko Tietze 2023-11-22 08:09:18 UTC Comment hidden (off-topic)
Comment 13 Michael Weghorn 2023-11-27 14:15:26 UTC Comment hidden (off-topic)
Comment 14 Heiko Tietze 2023-11-29 16:15:00 UTC Comment hidden (off-topic)
Comment 15 Michael Weghorn 2023-12-01 10:33:44 UTC Comment hidden (off-topic)
Comment 16 Heiko Tietze 2023-12-01 13:21:46 UTC Comment hidden (off-topic)
Comment 17 Michael Weghorn 2023-12-01 20:57:08 UTC Comment hidden (off-topic)
Comment 18 Stefan_Lange_KA@T-Online.de 2023-12-01 21:44:07 UTC
Question: My original problem was/is that a change of setting "Keep ratio" for images is not correctly handled at Undo/Redo.
Is the change of UI necessary to solve this problem resp. is it easier to solve the problem when the UI is changed?
Comment 19 Telesto 2023-12-02 02:29:33 UTC
(In reply to Stefan_Lange_KA@T-Online.de from comment #18)
> Question: My original problem was/is that a change of setting "Keep ratio"
> for images is not correctly handled at Undo/Redo.
> Is the change of UI necessary to solve this problem resp. is it easier to
> solve the problem when the UI is changed?

I don't think so. It appears to me that the bug here got hijacked by mistake for a different purpose.. IMHO

The "Improve UI in Image Properties Dialog for the locked state" surely liked, though. I'm unable to find the proper bug. Bug 144195 comment 8 is related and I might asked for lock button it up in context of table column sizes.
Comment 20 Stefan_Lange_KA@T-Online.de 2023-12-02 11:12:37 UTC
(In reply to Telesto from comment #19)
> (In reply to Stefan_Lange_KA@T-Online.de from comment #18)
> > Question: My original problem was/is that a change of setting "Keep ratio"
> > for images is not correctly handled at Undo/Redo.
> > Is the change of UI necessary to solve this problem resp. is it easier to
> > solve the problem when the UI is changed?
> 
> I don't think so. It appears to me that the bug here got hijacked by mistake
> for a different purpose.. IMHO
> 
> The "Improve UI in Image Properties Dialog for the locked state" surely
> liked, though. I'm unable to find the proper bug. Bug 144195 comment 8 is
> related and I might asked for lock button it up in context of table column
> sizes.

The title (summary) of this bug originally was "Undo/redo for checking/unchecking KeepRatio in Image Properties Dialog not properly handled". It was changed on 2023-11-22 08:09:18 UTC.
Comment 21 Heiko Tietze 2023-12-04 14:03:01 UTC
(In reply to Telesto from comment #19)
> It appears to me that the bug here got hijacked...
Harsh words. The ratio on/off toggle never was and will not be part of the image/object attribute. The "hi-jacking" aims to make this more clear - but yes, it will not change to be UI only. No undo/redo.
Comment 22 Stefan_Lange_KA@T-Online.de 2023-12-04 23:12:15 UTC
(In reply to Heiko Tietze from comment #21)
> (In reply to Telesto from comment #19)
> > It appears to me that the bug here got hijacked...
> Harsh words. The ratio on/off toggle never was and will not be part of the
> image/object attribute. The "hi-jacking" aims to make this more clear - but
> yes, it will not change to be UI only. No undo/redo.

It is correct: The KeepRatio setting for images isn't a part of the image object and - correctly - it isn't saved in odf subdocument "Content.xml".
But it is a setting - currently - valid for the whole document and saved as an entry in the odf subdocument "Settings.xml". IMHO therefore the change of this setting should be correctly handled regarding as well Undo and Redo as the internal switch "document changed" + reminder question "Document changed, close?".
Comment 23 Telesto 2023-12-05 03:30:51 UTC
(In reply to Heiko Tietze from comment #21)
> (In reply to Telesto from comment #19)
> > It appears to me that the bug here got hijacked...
> Harsh words. 

FWIW: Not intended to be harsh (or personal). Only to be pointy/direct. It has familiarity with https://en.scratch-wiki.info/wiki/Thread_Hijacking if you ask me. Changing te bug title and scope of a bug without prior notice and/or clarification

It can be pretty confusing and/or utterly astonishing for a bug reporter if the bug suddenly being about something different or focussing on a very narrow element of the original report
Comment 24 Heiko Tietze 2023-12-05 05:22:11 UTC
(In reply to Telesto from comment #23)
> It can be pretty confusing and/or utterly astonishing for a bug reporter if
> the bug suddenly being about something different or focussing on a very
> narrow element of the original report
Created bug 158531 and vote for WF here.

(In reply to Stefan_Lange_KA@T-Online.de from comment #22)
> ...saved as an entry in the odf subdocument "Settings.xml".
We shouldn't save this option then.
Comment 25 Telesto 2023-12-05 09:38:11 UTC
I think a fresh bug report is needed. The problems have mutated/evolved over time so the initial bug report doesn't match comment 0

(In reply to Stefan_Lange_KA@T-Online.de from comment #9)
> As far I have seen until now and just tested newly the setting "Keep ratio"
> from the Image properties dialog is used for all images (!) of a document,
> means one sets this value once on or off and this setting is used for all
> images. 

True, and undesired in my opinion. You enable something like this for each image individually. It surely shouldn't behave as a global setting. You open a image specific dialog and 'keep ratio' is applied for all images in document.. 

Note: The same behaviour is also observed in draw

If Keep Ratio should be toggled on or off by default another question and be saved for each image independently (if you ask me) and frame for that matter

> Furthermore this setting is saved in the odt subdocument
> "Settings.xml" as config item, e.g. <config:config-item
> config:name="KeepRatio" config:type="boolean">true</config:config-item>.
> So the value is kept when the document is closed and opened again. 

Not OK, should be a image specific setting. Which is my personal preference. Or nothing should be saved at all

> On the other hand the properties dialog e.g. for frames looking nearly - or
> completely (?) - like the properties dialog for images also contains the
> setting "Keep ratio" but when it is set on it is valid only for this one
> call of the dialog. It is lost at the next dialog call for this frame or
> other frames and it is also not used for images.
> Vice versa the "Keep ratio" value set for images is not used for frames.

Not storing the keep ratio checkbox is at least coherent. 

Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 5682e1d4145c26fc8021879df0543d5aeacf9c83
CPU threads: 8; OS: macOS 13.4.1; UI render: Skia/Raster; VCL: osx
Locale: nl-NL (nl_NL.UTF-8); UI: en-US
Calc: threaded
Comment 26 Stefan_Lange_KA@T-Online.de 2023-12-06 14:12:52 UTC
(In reply to Telesto from comment #25)
> I think a fresh bug report is needed. The problems have mutated/evolved over
> time so the initial bug report doesn't match comment 0
> 
> (In reply to Stefan_Lange_KA@T-Online.de from comment #9)
> > As far I have seen until now and just tested newly the setting "Keep ratio"
> > from the Image properties dialog is used for all images (!) of a document,
> > means one sets this value once on or off and this setting is used for all
> > images. 
> 
> True, and undesired in my opinion. You enable something like this for each
> image individually. It surely shouldn't behave as a global setting. You open
> a image specific dialog and 'keep ratio' is applied for all images in
> document.. 
> 
> Note: The same behaviour is also observed in draw
> 
> If Keep Ratio should be toggled on or off by default another question and be
> saved for each image independently (if you ask me) and frame for that matter
> 
> > Furthermore this setting is saved in the odt subdocument
> > "Settings.xml" as config item, e.g. <config:config-item
> > config:name="KeepRatio" config:type="boolean">true</config:config-item>.
> > So the value is kept when the document is closed and opened again. 
> 
> Not OK, should be a image specific setting. Which is my personal preference.
> Or nothing should be saved at all
> 
> > On the other hand the properties dialog e.g. for frames looking nearly - or
> > completely (?) - like the properties dialog for images also contains the
> > setting "Keep ratio" but when it is set on it is valid only for this one
> > call of the dialog. It is lost at the next dialog call for this frame or
> > other frames and it is also not used for images.
> > Vice versa the "Keep ratio" value set for images is not used for frames.
> 
> Not storing the keep ratio checkbox is at least coherent. 
> 
> Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
> Build ID: 5682e1d4145c26fc8021879df0543d5aeacf9c83
> CPU threads: 8; OS: macOS 13.4.1; UI render: Skia/Raster; VCL: osx
> Locale: nl-NL (nl_NL.UTF-8); UI: en-US
> Calc: threaded

I also think this bug is worn-out. But because there are several problems I am not sure if a new bug should be reported or better a proposal for enhancement:
- In the ODF Standard V 1.3 there is no image property "Keep aspect ratio". Even if in LibreOffice an individual KeepRatio value for image objects is introduced this setting cannot be saved as image property in a odf document.
- Propably it would be better when the "KeepRatio" setting is used only once as it is done in properties dialog for frames.
- The other question is how this setting is set initially or as default. IMHO this can be done by the "KeepRatio" value from Settings.xml, and this value could be set by a new Options value instead to take it from a Picture properties dialog.
Comment 27 Telesto 2023-12-06 15:04:40 UTC
@Regina
Any opinion on comment 26?

(In reply to Stefan_Lange_KA@T-Online.de from comment #26)
> I also think this bug is worn-out. But because there are several problems I
> am not sure if a new bug should be reported or better a proposal for
> enhancement:
> - In the ODF Standard V 1.3 there is no image property "Keep aspect ratio".
> Even if in LibreOffice an individual KeepRatio value for image objects is
> introduced this setting cannot be saved as image property in a odf document.
> - Propably it would be better when the "KeepRatio" setting is used only once
> as it is done in properties dialog for frames.
> - The other question is how this setting is set initially or as default.
> IMHO this can be done by the "KeepRatio" value from Settings.xml, and this
> value could be set by a new Options value instead to take it from a Picture
> properties dialog.
Comment 28 Armondo Lopez 2024-04-13 21:33:15 UTC
Just chiming in to say that this behavior is still present in

Version: 24.2.1.2 (X86_64) / LibreOffice Community
Build ID: db4def46b0453cc22e2d0305797cf981b68ef5ac
CPU threads: 8; OS: Windows 10.0 Build 19045; UI render: default; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: threaded

In this version, however

Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: a2265e8faa099d9652efd12392c2877c2df1d1eb
CPU threads: 8; OS: Windows 10.0 Build 19045; UI render: default; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: threaded

the "Keep Ratio" option is no longer present.
Comment 29 Stefan_Lange_KA@T-Online.de 2024-04-16 13:20:01 UTC
I see the "Keep Ratio" setting no longer as a property that needs to be saved in the document [see also my comment #26]. Because the UI of Image properties dialog is changed in 24.8 (checkbox "Keep ratio" replaced by a lock icon) I propose to put no effort into solving this bug but to close it.
Comment 30 Heiko Tietze 2024-04-17 09:19:57 UTC
(In reply to Stefan_Lange_KA@T-Online.de from comment #29)
> I see the "Keep Ratio" setting no longer as a property that needs to be
> saved in the document [see also my comment #26]. Because the UI of Image
> properties dialog is changed in 24.8 (checkbox "Keep ratio" replaced by a
> lock icon) I propose to put no effort into solving this bug but to close it.

Happy you appreciate the modification.
Comment 31 Telesto 2024-04-20 18:34:36 UTC
(In reply to Stefan_Lange_KA@T-Online.de from comment #29)
> I see the "Keep Ratio" setting no longer as a property that needs to be
> saved in the document [see also my comment #26]. Because the UI of Image
> properties dialog is changed in 24.8 (checkbox "Keep ratio" replaced by a
> lock icon) I propose to put no effort into solving this bug but to close it.

Are you sure:

1. open attached document "Test_Bildgröße_V2_keep_ratio_off.odt"
2. select the image and open the properties dialog -> KeepRatio is unchecked
3. check KeepRatio + exit by OK 
4. Result: the Undo arrow (gray; disabled)/ Redo arrow (gray: disabled) No red point is added to the Save icon 
5. Save
6. select the image and open the properties dialog again-> KeepRatio is still checked -> So apparently saved (as far I can tell)
Comment 32 Buovjaga 2024-04-21 06:41:30 UTC
Back to NEW per clarification.
Comment 33 Stefan_Lange_KA@T-Online.de 2024-04-21 18:36:37 UTC
(In reply to Telesto from comment #31)
> (In reply to Stefan_Lange_KA@T-Online.de from comment #29)
> > I see the "Keep Ratio" setting no longer as a property that needs to be
> > saved in the document [see also my comment #26]. Because the UI of Image
> > properties dialog is changed in 24.8 (checkbox "Keep ratio" replaced by a
> > lock icon) I propose to put no effort into solving this bug but to close it.
> 
> Are you sure:
> 
> 1. open attached document "Test_Bildgröße_V2_keep_ratio_off.odt"
> 2. select the image and open the properties dialog -> KeepRatio is unchecked
> 3. check KeepRatio + exit by OK 
> 4. Result: the Undo arrow (gray; disabled)/ Redo arrow (gray: disabled) No
> red point is added to the Save icon 
> 5. Save
> 6. select the image and open the properties dialog again-> KeepRatio is
> still checked -> So apparently saved (as far I can tell)

It is true: This behavior can be reproduced with LO 24.2 and also with LO 24.8 (changed UI: Lock symbol instead a checkbox)), means the original problem was not solved. 
But as I have written in Comment 29 I see "Keep Ratio" no longer as a document property necessary to be saved in the document because it doesn't influence the appearance of the document. I see it as a setting that makes easier to change image properties Width and Height in the Image properties dialog. This is like e.g. the autocorrection options  (Ignore dubble spaces etc.), also not saved within the document (AFAIK). Therefore I have proposed to close the bug.

On the other hand remains the question how this setting should be set initially or as default when the Image properties dialog is called.
One way could be to use always a default value - for images e.g. "Height Widht ratio is locked" ("Keep ratio" checked) - means the choosen setting is used only once . An other way could be to keep this setting for further calls and possibly to save it anywhere, e.g. in "Settings.xml" how is realized currently (then valid only for this document) or within the LO options. 
Nevertheless I see no need to mark the document as changed and to activate "Undo" when the setting is changed.
See also my Comment 26!

I think this cannot be object of a bug report but e.g a proposal for enhancement.
Comment 34 Heiko Tietze 2024-04-29 13:20:32 UTC
Miklos, you advice please. KeepRatio is indeed stored in settings.xml.
Comment 35 Miklos Vajna 2024-04-29 14:59:51 UTC
I'm afraid I don't have more thoughts other than the above comment 7
Comment 36 Heiko Tietze 2024-04-30 06:27:59 UTC
So back to the roots: Keep ratio is not considered for the undo stack. => bug
Comment 37 Telesto 2024-04-30 10:29:54 UTC
(In reply to Miklos Vajna from comment #35)
> I'm afraid I don't have more thoughts other than the above comment 7

I might have some issues interpreting your comment. Are you only giving a neutral observation. Or also entail a implicit conclusion, about which part being the bug; resulting in comment 36

-> "keep ratio" can be a special value for width or height in case the other axis has a fixed value.

Means by example: height being fixed value, when width changes and visa versa. 

-> "keep ratio" can be a view option, so if you enabled it for one image, there is some effort to enable it for an other image as well, if you adjust the width or height.

Means: 'Keep Ratio' not being a image specific setting, but a general setting. 
It will apply for all other images in document; 

A) this is a kind of odd, because the other controls in the image dialog apply to the specific image
B) Saving the Keep Ratio is not helpful in general. Assuming Keep Ratio being set the default. Implying the intention is that the default applying for each image individually until configured otherwise. This wouldn't work, because it's turning Keep Ratio of for a single image would apply to all other images. The default doesn't stick in the way the end-user would expect (or at least me)

I would see the point of saving it for each individual image. What MS Word appears to be doing also; but not being supported by Writer. Would improve document compatibility too. However no idea about the ODF side of the matter.

In the mean time, in tend to opt for Keep Ratio being session only thing. Disabled by default when opening a document. Re-using the previous state (checked/unchecked) when modified during a session. So the same behaviour as right now, except not saving the KeepRatio setting.
Comment 38 Stefan_Lange_KA@T-Online.de 2024-05-02 09:46:09 UTC
(In reply to Telesto from comment #37)
> (In reply to Miklos Vajna from comment #35)
> > I'm afraid I don't have more thoughts other than the above comment 7
> 
> I might have some issues interpreting your comment. Are you only giving a
> neutral observation. Or also entail a implicit conclusion, about which part
> being the bug; resulting in comment 36
> 
> -> "keep ratio" can be a special value for width or height in case the other
> axis has a fixed value.
> 
> Means by example: height being fixed value, when width changes and visa
> versa. 
> 
> -> "keep ratio" can be a view option, so if you enabled it for one image,
> there is some effort to enable it for an other image as well, if you adjust
> the width or height.
> 
> Means: 'Keep Ratio' not being a image specific setting, but a general
> setting. 
> It will apply for all other images in document; 
> 
> A) this is a kind of odd, because the other controls in the image dialog
> apply to the specific image
> B) Saving the Keep Ratio is not helpful in general. Assuming Keep Ratio
> being set the default. Implying the intention is that the default applying
> for each image individually until configured otherwise. This wouldn't work,
> because it's turning Keep Ratio of for a single image would apply to all
> other images. The default doesn't stick in the way the end-user would expect
> (or at least me)
> 
> I would see the point of saving it for each individual image. What MS Word
> appears to be doing also; but not being supported by Writer. Would improve
> document compatibility too. However no idea about the ODF side of the matter.
> 
> In the mean time, in tend to opt for Keep Ratio being session only thing.
> Disabled by default when opening a document. Re-using the previous state
> (checked/unchecked) when modified during a session. So the same behaviour as
> right now, except not saving the KeepRatio setting.

-> In main this my opinion (see Comment 33) but I wouldn't disable this setting for images by default but enable it to not distort the images when measures are changed. Not to keep ration between width and height should be the exceptionel case - and when possible for only one call of the properties dialog.