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 (
/).