Attachments (4)
Change History (24)
by , 3 years ago
Attachment: | 21408.patch added |
---|
comment:1 by , 3 years ago
Milestone: | → 21.09 |
---|---|
Owner: | changed from | to
Status: | new → assigned |
comment:2 by , 3 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:4 by , 3 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Yes, thank you a lot.
I am still testing so this is just my first feedback but I found some issues:
Major:
- with two objects with different values selected, adding another value results in adding
<different>;
+ the new value as value.
Minor:
- please, hide the number in parentheses at the end of the value if only one object is selected
- icons in the value list vanish if all objects have the same values
Not new, we miss a stronger indication for different values with checks. Right now it is enabled even if the tag is only present on some selected objects. With multiple values there is a small indication but I'd rather like to see a question mark ?
or something else but not a check.
Some of the new Jenkins warnings might be worth fixing.
follow-up: 6 comment:5 by , 3 years ago
Regarding row=""
, I still get scrollbars if the last value's icon is too tall. Probably icon height is not taken into account, yet.
comment:6 by , 3 years ago
Replying to skyper:
Regarding
row=""
, I still get scrollbars if the last value's icon is too tall. Probably icon height is not taken into account, yet.
An example is the tower:type
value list in the "General preset Mast" in Presets/MastAndTower.
follow-up: 8 comment:7 by , 3 years ago
I don't like it myself how <multiselect> works in conjunction with multiple selected primitives. It is very difficult to understand. It would probably be better to disable it if multiple primitives are selected.
Eg. if you select <different> + another value it does not add that value to what is already present on the selected primitives. It cannot do that because we can only write the same value to all selected primitives. (A limitation of the command interface.) What it can do is to overwrite all selected primitives with that value. I'm not sure it should, though.
As for the icon: it works for me. Maybe a skin issue? Which look&feel do you use?
comment:8 by , 3 years ago
Replying to marcello@…:
I don't like it myself how <multiselect> works in conjunction with multiple selected primitives. It is very difficult to understand. It would probably be better to disable it if multiple primitives are selected.
At least with different values, this might be better.
Eg. if you select <different> + another value it does not add that value to what is already present on the selected primitives. It cannot do that because we can only write the same value to all selected primitives. (A limitation of the command interface.) What it can do is to overwrite all selected primitives with that value. I'm not sure it should, though.
That works, too. Gives the user more options but also more responsibility.
As for the icon: it works for me. Maybe a skin issue? Which look&feel do you use?
I have tested with "Metal" and now "Dracula" from flatlaf plugin. With both the icons vanish.
- Install Presets/MastAndTower
- Add a new node, select it and open "Big communication tower" preset (only first letter a capital one) from the external preset.
The icon for tower:type=communication
is missing.
- Additionally, select
tower:type=observation
- Close Preset (
Ctrl+Return
) - Open Preset again
Now both icons are missing.
by , 3 years ago
Attachment: | 21408-2.patch added |
---|
Fixes vanishing icons, and scrollpane size if icons have different sizes. Also disables multiselect if the selected primitives have different values.
comment:10 by , 3 years ago
@marcello: this {2}List
is weird, is it a copy-paste error?
Logging.error(tr("Broken tagging preset \"{0}-{1}\" - number of items in ''{2}List'' must be the same as in ''values''", key, text, name));
comment:16 by , 3 years ago
The taggingpreset.fill-default-for-tagged-primitives
setting stopped working in r18254.
comment:17 by , 3 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
In 18254/josm: