Modify

Opened 13 years ago

Closed 12 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 bastiK)

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)

patch.diff (3.7 KB ) - added by Casiope 13 years ago.
New selection window.png (220.6 KB ) - added by Casiope 13 years ago.
6777.v2.patch (1.3 KB ) - added by simon04 13 years ago.
overflow.png (11.5 KB ) - added by simon04 13 years ago.
6777.v3.patch (1.4 KB ) - added by simon04 13 years ago.

Download all attachments as: .zip

Change History (17)

by Casiope, 13 years ago

Attachment: patch.diff added

by Casiope, 13 years ago

Attachment: New selection window.png added

comment:1 by bastiK, 13 years ago

Summary: Improve the selection window[patch] Improve the selection window

in reply to:  description comment:2 by simon04, 13 years ago

Replying to Casiope:

I made a very bad patch (I have any skill in Java) only to show what change is intended. So I need some help. :)

You can/should use trn() instead of tr() to handle singular/plural forms correctly.

To hide 0 nodes/ways/relations, you could try to construct the String iteratively:

  • start with tr("Selection: ")
  • add "{0} nodes" if the number of nodes is >0 (make use of trn() here)
  • similar for ways and relations

Think of something to insert the separators (/).

Last edited 13 years ago by simon04 (previous) (diff)

comment:3 by Casiope, 13 years ago

As I said, I have no skill in Java. This patch is the best I can do. :)

by simon04, 13 years ago

Attachment: 6777.v2.patch added

by simon04, 13 years ago

Attachment: overflow.png added

in reply to:  3 comment:4 by simon04, 13 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 bastiK, 13 years ago

To prevent long text, "Relation" can be abbreviated (like it is done at the moment).

by simon04, 13 years ago

Attachment: 6777.v3.patch added

comment:6 by simon04, 13 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 stoecker, 13 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 simon04, 13 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? ;-)

comment:9 by bastiK, 13 years ago

Description: modified (diff)

What about 8 translatable strings for every combination?

in reply to:  9 ; comment:10 by stoecker, 13 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.

in reply to:  10 comment:11 by bastiK, 13 years ago

Replying to stoecker:

Replying to bastiK:

What about 8 translatable strings for every combination?

Wont work. There are languages which have up to 6 plural forms.

Ouch, this doesn't seem to be trivial at all.

comment:12 by stoecker, 12 years ago

Resolution: wontfix
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.