Hi Peppe,

I just committed what seems to fix the problem of the COGO command panel.

I created a new class for the Frame containing the commandPanel.
The frame is created only once when the EditToolBox is initialized (it is invisible at creation time),
Then it is made visible :
- if the user click on the CogoTool "button"
And it is made invisible :
- if the user close the InternaleFrame
- if the user change the activeCursorTool (indeed, when the CogoTool "button" state changes to unselected)

I also changed the relative position of the polygon check box and the commands panel In the commandLineStringPanel
so that the commands panel can resize when the Frame is resized.

Let me know if you are OK with both changes,

Michaël

Le 23/03/2017 à 18:29, Michaël Michaud a écrit :

Hi Peppe,

I tried to fix the problem but could not yet.
The tool has another flaw as it is not possible to add the first point from the console.

Maybe instead of creating the commandDialog in the CursorTool, we should create it in the EditToolboxCADPlugIn and open it / make it visible from the button associated to the CursorTool (adding an actionListener to the button). (just an idea, I think aloud, maybe I'll try this week end but I'm not sure I'll have the time)

 Michaël

Le 20/03/2017 à 13:37, Giuseppe Aruta a écrit :
The class involved on that is:
org.openjump.advancedtools.tools.cogo.DrawGeometryCommandsTool
I tried a method ( still in the class, line 178 - closeAction()) to finalize the tool and to activate SelectTool but it won't work
Peppe

2017-03-20 13:26 GMT+01:00 Giuseppe Aruta <giuseppe.ar...@gmail.com <mailto:giuseppe.ar...@gmail.com>>:

    Hi Ede
    1) open CAD toolbox
    2) click on Draw woth command tool
    3) draw something using commands or directly on the workbench
    4) click on the red cross of the dialog

    In this case the command tool should be deactivated (and maybe
    the selection tool should be activated).

    On the contrary the command tool is still active and the COGO
    dialog comes out after the first click on the workbench

    Peppe

    2017-03-20 13:09 GMT+01:00 <edgar.sol...@web.de
    <mailto:edgar.sol...@web.de>>:

        On 20.03.2017 13:03, Giuseppe Aruta wrote:
        > Hi
        > 1) I still have to finish Spanish translation for Cad
        toolbox and
        > 2) there is still the COGO issue pointed out by Michael.
        > I can work on 1) on next WE. For 2) I ask a help to
        minimize the time.

        wrt. 2.
        i can't reproduce it. Mike?

        > Apart these issues I agree for 1:11
        > Best regards
        > Peppe

        ok, i'll wait for your go! tell me if i can help somehow.. ede

        >
        >
        >
        >
        > Il 20/Mar/2017 12:23, "Rahkonen Jukka (MML)"
        <jukka.rahko...@maanmittauslaitos.fi
        <mailto:jukka.rahko...@maanmittauslaitos.fi>
        <mailto:jukka.rahko...@maanmittauslaitos.fi
        <mailto:jukka.rahko...@maanmittauslaitos.fi>>> ha scritto:
        >
        >     JTS has a new home and new license JTS
        https://www.locationtech.org/projects/technology.jts
        <https://www.locationtech.org/projects/technology.jts>
        <https://www.locationtech.org/projects/technology.jts
<https://www.locationtech.org/projects/technology.jts>> but Michaël wrote in a comment here
        https://sourceforge.net/p/jump-pilot/bugs/417/
        <https://sourceforge.net/p/jump-pilot/bugs/417/>
        <https://sourceforge.net/p/jump-pilot/bugs/417/
        <https://sourceforge.net/p/jump-pilot/bugs/417/>> that the
        change would be big due to changed package names.
        >
        >     -Jukka-
        >     ________________________________________
        >     Lähettäjä: edgar.sol...@web.de
        <mailto:edgar.sol...@web.de> <mailto:edgar.sol...@web.de
        <mailto:edgar.sol...@web.de>> <edgar.sol...@web.de
        <mailto:edgar.sol...@web.de> <mailto:edgar.sol...@web.de
        <mailto:edgar.sol...@web.de>>>
        >     Lähetetty: 20. maaliskuuta 2017 13:09
        >     Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
        <mailto:jump-pilot-devel@lists.sourceforge.net>
        <mailto:jump-pilot-devel@lists.sourceforge.net
        <mailto:jump-pilot-devel@lists.sourceforge.net>>
        >     Aihe: Re: [JPP-Devel] next release options
        >
        >     On 20.03.2017 08:35, Rahkonen Jukka (MML) wrote:
        >     > use the Locationtech version of JTS if it makes sense.
        >
        >     what's that:)  and why would it make sense? ..ede
        >
> ------------------------------------------------------------------------------
        >     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
        <mailto:Jump-pilot-devel@lists.sourceforge.net>
        <mailto:Jump-pilot-devel@lists.sourceforge.net
        <mailto:Jump-pilot-devel@lists.sourceforge.net>>
        >
        https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
        <https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel>
        <https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
        <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
        <mailto:Jump-pilot-devel@lists.sourceforge.net>
        <mailto:Jump-pilot-devel@lists.sourceforge.net
        <mailto:Jump-pilot-devel@lists.sourceforge.net>>
        >
        https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
        <https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel>
        <https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
        <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
        <mailto:Jump-pilot-devel@lists.sourceforge.net>
        >
        https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
        <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
        <mailto:Jump-pilot-devel@lists.sourceforge.net>
        https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
        <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


------------------------------------------------------------------------------
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