Modify

Opened 9 years ago

Closed 9 years ago

#12215 closed enhancement (fixed)

The components of Conflation Plugin's settings dialog are misaligned

Reported by: coso Owned by: team
Priority: normal Milestone:
Component: Plugin conflation Version:
Keywords: Cc:

Description

The components of Conflation Plugin's settings dialog are misaligned.

I provided a patch as pull request here that does the following changes:

  • align components to the left and add regular spaces among them
  • add a "Layer:" label to explain the meaning of "(none)" string
  • add icons to buttons
  • translate the strings with the numbers of selected objects
  • fix typo: "subject" --> "reference" in SettingsDialog.java#L215

See the pull request for before/after screenshot.

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2015-11-24 00:04:12 +0100 (Tue, 24 Nov 2015)
Build-Date:2015-11-23 23:14:21
Revision:9060
Relative:URL: ^/trunk

Identification: JOSM/1.5 (9060 it) Linux Ubuntu 15.10
Memory Usage: 329 MB / 876 MB (258 MB allocated, but free)
Java version: 1.8.0_66-internal, Oracle Corporation, OpenJDK 64-Bit Server VM
VM arguments: [-Djosm.restart=true, -Djava.net.useSystemProxies=true]

Plugins:
- conflation (0.1.9)
- jts (31772)
- utilsplugin2 (31772)

Attachments (0)

Change History (5)

comment:1 by coso, 9 years ago

I have a question.
I though to report the pull request here too, just for the records, but maybe this wasn't necessary.
Should I create a ticket the next time I propose a pull request or add an issue about a plugin in JOSM organization on GitHub?

comment:2 by wiktorn, 9 years ago

This should be not necessary, as the pull request should be handled by the maintainer and as a part of this process, he should be able to update repository with new *jar file.

comment:3 by Don-vip, 9 years ago

thanks for the PR, I have merged it :)

I see conflation plugin is using a COTS: MigLayout. It is only used one time in MatchFinderPanel. I don't think the use of this COTS is justified and I'd like to see it removed, would you be interested to help us to do that in a new PR?

comment:4 by coso, 9 years ago

@wiktor
Thanks.

@Don-vip
Thank you for merging.
\o/ My first code contribution to JOSM (almost) :-)

I'm new to Java but it doesn't seem difficult to remove MigLayout. I'll take a look at that.

comment:5 by Don-vip, 9 years ago

Resolution: fixed
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.