Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#14840 closed enhancement (fixed)

[patch] Add separation borders to search screen

Reported by: jidanni Owned by: bafonins
Priority: minor Milestone: 17.06
Component: Core Version:
Keywords: search Cc: bafonins

Description (last modified by Don-vip)

Please add some borders as show below:


Attachments (11)

josmSearch.jpg (29.0 KB ) - added by jidanni 7 years ago.
search-old.jpg (110.2 KB ) - added by bafonins 7 years ago.
search-expert-new.jpg (109.3 KB ) - added by bafonins 7 years ago.
search-new.jpg (54.8 KB ) - added by bafonins 7 years ago.
ticket14840.patch (15.0 KB ) - added by bafonins 7 years ago.
Refactored Search Dialog
ticket14840_2.patch (12.2 KB ) - added by bafonins 7 years ago.
Merged supplied patch with the latest release of JOSM.
ticket14840_3.patch (11.8 KB ) - added by bafonins 7 years ago.
Resolved minor ant-checkstyle warnings
14840-v4.patch (11.8 KB ) - added by bafonins 7 years ago.
Just for convenience, merged two previous patched into one
Suche.png (31.4 KB ) - added by Klumbumbus 7 years ago.
s-dialog.jpg (131.7 KB ) - added by bafonins 7 years ago.
Fixed alignment
14840-alignment-fix.patch (1.7 KB ) - added by bafonins 7 years ago.
Sorry for that, this should fix the issue with the alignment

Download all attachments as: .zip

Change History (28)

by jidanni, 7 years ago

Attachment: josmSearch.jpg added

comment:1 by Don-vip, 7 years ago

Description: modified (diff)
Keywords: search added

by bafonins, 7 years ago

Attachment: search-old.jpg added

by bafonins, 7 years ago

Attachment: search-expert-new.jpg added

by bafonins, 7 years ago

Attachment: search-new.jpg added

comment:2 by bafonins, 7 years ago

Hi!

Spent some time on it, would like to show the results.
Here is the old version, view in the expert mode :

Here is the new version, again in the expert mode :

And finally, the regular mode :

In my opinion, looks more compact and structured. Any suggestions?

comment:3 by stoecker, 7 years ago

Missing the patch file.

"Search examples" is wrong, as only some are examples, most are ready to use clicks.

by bafonins, 7 years ago

Attachment: ticket14840.patch added

Refactored Search Dialog

comment:4 by Klumbumbus, 7 years ago

Summary: Add separation borders to search screen[patch] Add separation borders to search screen

comment:5 by stoecker, 7 years ago

Your patch contains lot of stuff which should not be there including reverts of changes done recently.

by bafonins, 7 years ago

Attachment: ticket14840_2.patch added

Merged supplied patch with the latest release of JOSM.

in reply to:  3 comment:6 by michael2402, 7 years ago

Cc: bafonins added

Replying to stoecker:

"Search examples" is wrong, as only some are examples, most are ready to use clicks.

Examples that are useful are the best ones ;-).

I think "hints" is a better word. Or "syntax" (but it is not a real syntax definition, so "hints on search syntax"?)

I can do the merging next week (after I figured out the backspace-bug)

Last edited 7 years ago by michael2402 (previous) (diff)

by bafonins, 7 years ago

Attachment: ticket14840_3.patch added

Resolved minor ant-checkstyle warnings

comment:7 by jidanni, 7 years ago

Nice design.

by bafonins, 7 years ago

Attachment: 14840-v4.patch added

Just for convenience, merged two previous patched into one

comment:8 by michael2402, 7 years ago

Resolution: fixed
Status: newclosed

In 12333/josm:

Fix #14840: Add separation borders to search dialog. Patch by bafonins

comment:9 by Don-vip, 7 years ago

Milestone: 17.06

comment:10 by Don-vip, 7 years ago

Priority: trivialminor

comment:11 by Don-vip, 7 years ago

Nice, thanks! :)

by Klumbumbus, 7 years ago

Attachment: Suche.png added

comment:12 by Klumbumbus, 7 years ago

In localized JOSM it looks a bit strange. Could the text of the three blocks on the left made left aligned?


comment:13 by Klumbumbus, 7 years ago

Wouldn't "Syntax" be a better name for the block "Search Options"?

comment:14 by Klumbumbus, 7 years ago

Resolution: fixed
Status: closedreopened

comment:15 by michael2402, 7 years ago

Owner: changed from team to bafonins
Status: reopenednew

by bafonins, 7 years ago

Attachment: s-dialog.jpg added

Fixed alignment

by bafonins, 7 years ago

Attachment: 14840-alignment-fix.patch added

Sorry for that, this should fix the issue with the alignment

comment:16 by michael2402, 7 years ago

Resolution: fixed
Status: newclosed

In 12345/josm:

Apply #14840: Fix alignment in search dialog. Patch by bafonins

comment:17 by jidanni, 7 years ago

Thanks.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain bafonins.
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.