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 , 9 years ago
comment:2 by , 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 , 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 , 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 , 9 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
MigLayout removed with https://github.com/JOSM/josm-conflation-plugin/pull/2 thanks! :)
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?