search

glyphr-studio / Glyphr-Studio-2

Glyphr Studio v1 is more than a decade old. V2 is here to save the day.
https://www.glyphrstudio.com/v2
116 stars 14 forks source link

Glyph Editor: "Lock aspect ratio" is not respected in bulk-edit for paths #154

Closed ThePaSch closed 6 months ago

ThePaSch commented 7 months ago

Describe the bug & what the expected behavior should be: When attempting to bulk-edit paths for a given glyph, the "Lock aspect ratio" option is not respected. When said option is ticked, manually adjusting the height or width should proportionally adjust the other respective value; however, this is currently not the case.

Steps to reproduce the bug:

  1. Create a new project.
  2. Create a new glyph for any character.
  3. Add a new path for said glyph; add any number of points, then apply.
  4. Deselect that path.
  5. Tick the "lock aspect ratio" box in the attribute editor's "Bulk-edit paths" section.
  6. Adjust either width or height in that section.
  7. The other respective value is not adjusted accordingly.
mattlag commented 7 months ago

Thank you for filing the bug! I was able to reproduce it.

One note is that the on-canvas handle adjustments do honor the locked aspect ratio... but not the width/height inputs.

mattlag commented 7 months ago

I'm still looking into this one... it seems like it only happens sometimes, but sometimes it works as expected. There is also another hard to reproduce bug with the Number Input control while using the keyboard arrow keys to increment/decrement... It seems to multiply instead of just increment.

Anyway, I tried to include this in 2.0.1, but it's a bit more complex than I thought.

mattlag commented 6 months ago

Fixed in v2.1.1