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

Two new scales when zooming : max scale for points or other geom #607

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

Gustry
Copy link
Member

@Gustry Gustry commented Oct 14, 2024

image

  • Funded by: CC Bièvre Est
  • Description:
    • Two new scales when zooming : max scale for points or other geom

If the value in the widget is the same as the default, it's not written in the CFG file, to avoid not needed keys.
@nboisteault What do you think 5000.0 ?

@nboisteault
Copy link
Member

@Gustry scale values are not kept in the UI

simplescreenrecorder-2024-10-21_10.28.17.mp4

@Gustry
Copy link
Member Author

Gustry commented Oct 21, 2024

Sorry @nboisteault It's because we are using the "scale widget" for the first time in the plugin.
It's fixed now.

@nboisteault
Copy link
Member

@Gustry in the cfg I have "max_scale_lines_polygons": 25000.0,. Is it needed to have the point and 0?

@Gustry
Copy link
Member Author

Gustry commented Oct 21, 2024

The QGIS API returns a double value :
https://api.qgis.org/api/classQgsScaleWidget.html#a957b064d233dad36fd5eb03cee2c05ae

It's the raw value, from the scale widget.

The scale value indicates the scale denominator, e.g. 1000.0 for a 1:1000 map.

Is-it blocking on your side ? Do you know the decimal separator ? Or do you want me to cast to int ?

@Gustry
Copy link
Member Author

Gustry commented Oct 21, 2024

I also planning to update from these "Minimum scale" and "Maximum scale" text widgets to this scale widget, so I should be careful to cast to int in the CFG ?

@Gustry Gustry merged commit 44a6c9c into 3liz:master Oct 29, 2024
5 checks passed
@Gustry Gustry deleted the max-scale-geom branch October 29, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants