Setting Font size in label dialog not working
I'm sure this has to do with the making of the font size dynamic, so I guess we need to decide whether to even allow the font size to be changed. Perhaps change it to a minimum font size setting?
Use case: Select View->Labels..., Change font size from 9 to 12, click OK. Expected Result: Font size gets larger. Observed result: Font size remains at 9.
There is plenty of room for a larger font in the view I was looking at when I attempted this:
Comments (11)
-
repo owner -
- changed milestone to General interface
- changed version to alpha02
-
repo owner -
assigned issue to
-
assigned issue to
-
After our discussion, the behavior of label sizes should be the following: by default, the labels should take as much space as available (depending on the size of the tile). In Label settings, there should be a checkbox ("Keep fixed") that would allow the user to specify a certain label size and make it insensitive to zoom level.
-
The general consensus was also that the current labels are too small, with respect to the available space. We should try to make them at least ~50% bigger.
-
repo owner - changed status to resolved
Changing this to resolved as the function to set a fixed label size has been implemented. Let me know if the dynamic resizing still gets the sizes wrong.
-
- changed status to open
Reopening this because noticed a few issues while testing:
1) The value set when checking "Keep fixed" works only until you zoom out enough to hide the labels. When you zoom back in, the label size is restored to the default 14 (although, the "keep fixed" setting keeps working).
2) When defined dynamically, the font size could still be much bigger than it is now. I'm attaching 2 screenshots taken at exactly the same zoom level: one is with dynamic font sizing, the other one with manual to show how big the labels can be at this tile size.
-
repo owner - changed status to resolved
Solved aforementioned issues and added the ability to specify min/ max ranges for label sizes.
-
reporter - changed status to closed
Merged to master.
-
reporter - changed component to General interface
-
reporter - removed milestone
Removing milestone: General interface (automated comment)
- Log in to comment
Yes, I already commented on this in
#175.