Opened 14 years ago
Closed 14 years ago
#6777 closed enhancement (wontfix)
[patch] Improve the selection window
Reported by: | Casiope | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | latest |
Keywords: | selection window | Cc: |
Description (last modified by )
The selection window can be improved very easily.
Nodes, ways and relations should use the singular and plural forms.
They should be hidden when they are not selected.
I made a very bad patch (I have any skill in Java) only to show what change is intended. So I need some help. :)
Thank you very much.
Attachments (5)
Change History (17)
by , 14 years ago
Attachment: | patch.diff added |
---|
by , 14 years ago
Attachment: | New selection window.png added |
---|
comment:1 by , 14 years ago
Summary: | Improve the selection window → [patch] Improve the selection window |
---|
follow-up: 4 comment:3 by , 14 years ago
As I said, I have no skill in Java. This patch is the best I can do. :)
by , 14 years ago
Attachment: | 6777.v2.patch added |
---|
by , 14 years ago
Attachment: | overflow.png added |
---|
comment:4 by , 14 years ago
Replying to Casiope:
As I said, I have no skill in Java. This patch is the best I can do. :)
I tried to give you some hints how to implement it. Just for the case that you would like to improve your skills. :-)
I attached a reworked patch (attachment:6777.v2.patch). Unfortunately, the text overflows when a lot of stuff is selected (cf. attachment:overflow.png).
comment:5 by , 14 years ago
To prevent long text, "Relation" can be abbreviated (like it is done at the moment).
by , 14 years ago
Attachment: | 6777.v3.patch added |
---|
comment:6 by , 14 years ago
What about "1 Rel., 97 Ways, 708 Nodes selected"
(see attachment:6777.v3.patch). This moves "selected" to the end, so a overflow first hides the obvious part …
comment:7 by , 14 years ago
You all only look at English strings. The current form is one string, which can be properly translated and fixed in multiple languages. The joined texts are much harder to translate and all assumptions about word lengths are invalid for any other language.
comment:8 by , 14 years ago
Without separating the primitive types, singular/plural text won't work, right?
I wonder how to get everything right for this ticket. In general, it is quite a long discussion for this trivial issue, isn't it? ;-)
follow-up: 10 comment:9 by , 14 years ago
Description: | modified (diff) |
---|
What about 8 translatable strings for every combination?
follow-up: 11 comment:10 by , 14 years ago
Replying to bastiK:
What about 8 translatable strings for every combination?
Wont work. There are languages which have up to 6 plural forms.
comment:11 by , 14 years ago
comment:12 by , 14 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Replying to Casiope:
You can/should use
trn()
instead oftr()
to handle singular/plural forms correctly.To hide 0 nodes/ways/relations, you could try to construct the String iteratively:
tr("Selection: ")
"{0} nodes"
if the number of nodes is >0 (make use oftrn()
here)Think of something to insert the separators (
/
).