Bug 496803 - plot padding settings issues
Summary: plot padding settings issues
Status: RESOLVED FIXED
Alias: None
Product: LabPlot2
Classification: Applications
Component: frontend (show other bugs)
Version: latest
Platform: Other Other
: NOR normal
Target Milestone: ---
Assignee: Stefan Gerlach
URL:
Keywords:
Depends on:
Blocks: 496804
  Show dependency treegraph
 
Reported: 2024-11-28 22:52 UTC by uwestoehr
Modified: 2024-12-17 09:49 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
screencast (270.46 KB, image/gif)
2024-11-28 22:52 UTC, uwestoehr
Details
LabPlot test file (36.73 KB, application/x-xz)
2024-12-16 02:32 UTC, uwestoehr
Details

Note You need to log in before you can comment on or make changes to this bug.
Description uwestoehr 2024-11-28 22:52:59 UTC
Created attachment 176197 [details]
screencast

Release build a79c86e, Nov 26 2024, 22:43:35

- open a file of your choice and go to a plot
- in the plot properties go to Layout and change the padding

result:
1. the default step size is 1e-5 cm, see the screencast. It should be 0.1 cm as less than one micron is not sensible in no use case
2. the option symmetric should in my opinion be the default as this is in most cases desired
3. the values should not have 5 decimals by default as the base unit is cm. The eye has a resolution ob about 50 µm thus anything less than 100 µm is not really visible
Comment 1 Stefan Gerlach 2024-12-11 22:13:23 UTC
Git commit a1b8549a60ab32ef3f2e151a5af788df28246efd by Stefan Gerlach.
Committed on 11/12/2024 at 22:07.
Pushed by sgerlach into branch 'master'.

opj: round padding when importing

M  +16   -0    src/backend/datasources/projects/OriginProjectParser.cpp

https://invent.kde.org/education/labplot/-/commit/a1b8549a60ab32ef3f2e151a5af788df28246efd
Comment 2 uwestoehr 2024-12-16 02:32:14 UTC
The bug is not fixed in 

Release build b8b4b1e
Dec 14 2024, 22:36:14

- symmetric is still not the default
- there are still 5 digits
- the increment is still 1e-5 cm

tested with the attached LabPlot file
Comment 3 uwestoehr 2024-12-16 02:32:54 UTC
Created attachment 176669 [details]
LabPlot test file
Comment 4 Stefan Gerlach 2024-12-17 09:46:28 UTC
Git commit 9d32b6e5d3a48123ec01161c5f181bff0820af29 by Stefan Gerlach.
Committed on 17/12/2024 at 09:45.
Pushed by sgerlach into branch 'master'.

Fix spinboxes for padding

M  +0    -14   src/backend/datasources/projects/OriginProjectParser.cpp
M  +5    -0    src/backend/lib/macros.h
M  +6    -0    src/frontend/dockwidgets/CartesianPlotDock.cpp
M  +59   -65   src/frontend/ui/dockwidgets/cartesianplotdock.ui

https://invent.kde.org/education/labplot/-/commit/9d32b6e5d3a48123ec01161c5f181bff0820af29
Comment 5 Stefan Gerlach 2024-12-17 09:48:43 UTC
- symmetric is still not the default
When opening a project, the saved value is used. Importing an OPJ file, symmetric is set when it is symmetric.
- there are still 5 digits
Now 1 digit is the default
- the increment is still 1e-5 cm
increment is the last digit