hey Mike,

just tried "layer context menu -> layer properties" and that works for me. 
maybe this is different on a two screen system which i do not have set up right 
now.

could you please try and send the stack if you can still reproduce it?

..ede

On 05.04.2018 22:25, Michaël Michaud wrote:
> Hi Ede,
> 
> The piece of code you commented was the only solution I found to fix the NPE 
> arising during opening of the DetachableInternalFrame created by 
> NewLayerProperties (layer contextual menu).
> 
> However, this is just a workaround. I first thought the internal frame was 
> missing a parent,
> but it is added to the WorkbenchFrame the same way as, for example, 
> ViewAttributesFrame in ViewAttributePlugIn :
> 
> context.getWorkbenchFrame().addInternalFrame(JInternalFrame, boolean, 
> boolean);
> 
> So I don't know why adding this particular internal frame is throwing a NPE.
> 
> If you find the proper way to fix NewLayerProperties, you're welcome, in the 
> meantime, your improvment in the task frames placement will re-introduce this 
> NPE.
> 
> Michaël 
> 
> 
> Le 05/04/2018 à 16:11, edgar.sol...@web.de a écrit :
>> hey Mike,
>>
>> i noticed that task frames were not properly placed into top left corner of 
>> the desktop pane but with some offset from the top. commenting the below 
>> change of you multi screen fix seems to resolve the issue.
>>
>> can you double check how or if the below was needed or can be removed?
>>
>> thx.. ede
>>
>>
>> -------- Forwarded Message --------
>> Subject: [JPP-Devel] SVN: [5734] 
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/ GUIUtil.java
>> Date: Thu, 05 Apr 2018 14:08:31 +0000
>> From: jump-pilot-svn--- via Jump-pilot-devel 
>> <jump-pilot-devel@lists.sourceforge.net>
>> Reply-To: OpenJump develop and use <jump-pilot-devel@lists.sourceforge.net>
>> To: jump-pilot-devel@lists.sourceforge.net
>> CC: jump-pilot-...@lists.sourceforge.net
>>
>> Revision: 5734
>>           http://sourceforge.net/p/jump-pilot/code/5734
>> Author:   edso
>> Date:     2018-04-05 14:08:31 +0000 (Thu, 05 Apr 2018)
>> Log Message:
>> -----------
>> fix TaskFra mes bot placed properly into left upper corner of desktop pane
>>
>> Modified Paths:
>> --------------
>>     core/trunk/src/com/vividsolutions/jump/workbench/ui/GUIUtil.java
>>
>> Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/GUIUtil.java
>> ===================================================================
>> --- core/trunk/src/com/vividsolutions/jump/workbench/ui/GUIUtil.java 
>> 2018-04-05 13:26:30 UTC (rev 5733)
>> +++ core/trunk/src/com/vividsolutions/jump/workbench/ui/GUIUtil.java 
>> 2018-04-05 14:08:31 UTC (rev 5734)
>> @@ -1348,7 +1348,7 @@
>>          (int) other.getLocationOnScreen().getY()
>>              + (location.fromBottom ? (other.getHeight()
>>                  - componentToMove.getHeight() - location.y) : location.y));
>> -    if (!(componentToMove instanceof Window) && !(componentToMove 
>> instanceof JInternalFrame)) {
>> +    if (!(componentToMove instanceof Window) /*&& !(componentToMove 
>> instanceof JInternalFrame) */) {
>>        SwingUtilities.convertPointFromScreen(p, componentToMove.getParent());
>>      }
>>      componentToMove.setBounds(p.x, p.y, width, height);
>>
>>
>> ------------------------------------------------------------------------------
>> Check out the vibrant tech community on one of the world's most
>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> Jump-pilot-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
> 


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to