Re: [josm-dev] Ticket #351: Edit Relation dialog is always on top

2008-08-15 Thread Dirk Stöcker
On Tue, 12 Aug 2008, Florian Heer wrote:

 attached patch solves #351, but I am not absolutely sure if that is a good 
 idea. The line
 try { setAlwaysOnTop(true); } catch (SecurityException sx) {}
 which had to be commented out for this, had been put there for a reason. I 
 don't see a reason, but maybe there is one.

Copied your patch into the bug-report. Don't know how to decide, so I move 
it into the future :-)

Ciao
-- 
http://www.dstoecker.eu/ (PGP key available)

___
josm-dev mailing list
josm-dev@openstreetmap.org
http://lists.openstreetmap.org/listinfo/josm-dev


[josm-dev] Ticket #351: Edit Relation dialog is always on top

2008-08-12 Thread Florian Heer

Hi again,

attached patch solves #351, but I am not absolutely sure if that is a 
good idea. The line

try { setAlwaysOnTop(true); } catch (SecurityException sx) {}
which had to be commented out for this, had been put there for a reason. 
I don't see a reason, but maybe there is one.


Regards, Florian.
Index: src/org/openstreetmap/josm/gui/dialogs/RelationEditor.java
===
--- src/org/openstreetmap/josm/gui/dialogs/RelationEditor.java  (revision 775)
+++ src/org/openstreetmap/josm/gui/dialogs/RelationEditor.java  (working copy)
@@ -153,7 +153,8 @@
as well as the members. In addition to this we should 
have a smart editor that  +
detects the type of relationship and limits your 
choices in a sensible way.)+/em/html);
getContentPane().add(help, BorderLayout.NORTH);
-   try { setAlwaysOnTop(true); } catch (SecurityException sx) {}
+// Is there an important reason for AlwaysOnTop? - Commented out by Florian 
Heer   
+// try { setAlwaysOnTop(true); } catch (SecurityException sx) {}

// Basic Editor panel has two blocks; 
// a tag table at the top and a membership list below.
___
josm-dev mailing list
josm-dev@openstreetmap.org
http://lists.openstreetmap.org/listinfo/josm-dev