On Wed, Jun 28, 2017 at 2:26 PM, Pascal via Gnoga-list <
gnoga-list@lists.sourceforge.net> wrote:

> Hello Jeremiah,
>
> > Le 25 juin 2017 à 16:45, Jeremiah Breeden <jeremiah.bree...@gmail.com>
> a écrit :
> >
> > Sorry for the lateness of this.  My kids have been keeping me busy as of
> late.  Attached are both versions.  I tested compilation by adding them to
> my local Gnoga library and using "make gnoga".  I tested functionality in
> various of my smaller projects (I'll attach a test program I used, but it
> isn't gnoga style compliant).  That said, I don't know how bullet proof
> they are yet.  If I had more time, I would have generated a lot of tests.
> To run my test program simply call it in your On_Connect handler ina
> multiconnect gnoga program like this:
> >
> >    procedure On_Connect
> >       (Main_Window : in out Gnoga.Gui.Window.Window_Type'Class;
> >        Connection  : access
> >           Gnoga.Application.Multi_Connect.Connection_Holder_Type)
> >    is
> >    begin
> >       Test_Dialog.On_Connect(Main_Window,Connection);
> >    end On_Connect;
> >
> >
> > Tested in Windows 10, GNAT GPL 2016, Firefox (current version)
>
> Thanks again for your contribution, however I prefer add a standalone test
> program than a package as it is easier to build and run.
> Here is the one I've made from your test package:
>
>
> Is it correct for you ?
>

This works for me.  Thanks.

>
> >
> > >    procedure Remove (Dialog : in out Dialog_Type);
> > >    --  Removes an element from the DOM, if the ID_Type is DOM_ID, the
> ID
> >
> > Removes the dialog?
> >
> > Yes, this maybe overkill.  It cannot be dynamically managed in version 2
> but you can still remove it from the DOM (if I did it correctly) in cases
> where you want to manually manage stuff.  Kind of a compromise.  any of
> this stuff is up for debate/removal.
>
> In fact, I was pointing the comment, Shouldn't it be "--  Removes dialog
> from the DOM..."?
>

I used the same comment found in the Gnoga library for consistency.
Normally when I override something, I try to do that so it makes someone
look closer.  I don't mind if you want to change it though.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Gnoga-list mailing list
Gnoga-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/gnoga-list

Reply via email to