assetsense / gwtwindowmanager

Automatically exported from code.google.com/p/gwtwindowmanager
0 stars 0 forks source link

GInternalFrameEventListener needs all functions to be defined #27

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Build a GInternalFrameEventListner; eg:
frame.addInternalFrameListener(new GInternalFrameLister() {

});
2. If you have a good IDE (like Eclipse) it will complain that none of the
functions are defined and that they need to be all defined.

What is the expected output? What do you see instead?
I would expect that only 1 is needed. I don't know how to design this but
it is rather annoying that you need to define all events when only needing 1.

What version of the product are you using? On what operating system?
0.5 gwt 1.3.3 wxp

Please provide any additional information below.

Original issue reported on code.google.com by robbie.d...@gmail.com on 15 Feb 2007 at 11:34

GoogleCodeExporter commented 8 years ago
You need an Adapter :) do this:

public class GInternalFrameAdapter implements GInternalFrameListener {

    public void frameResized(GInternalFrameEvent evt){}

    public void frameOpened(GInternalFrameEvent evt){}

    public void frameClosed(GInternalFrameEvent evt){}

    public void frameMaximized(GInternalFrameEvent evt){}

    public void frameMinimized(GInternalFrameEvent evt){}

    public void frameIconified(GInternalFrameEvent evt){}

    public void frameRestored(GInternalFrameEvent evt){}

    public void frameMoved(GInternalFrameEvent evt) { }

}

Now you only have to instanciate like this:
frame.addInternalFrameListener(new GInternalFrameAdapter(){
    implement just what you need...
});

Original comment by marceloemanoel on 15 Feb 2007 at 11:57

GoogleCodeExporter commented 8 years ago
Nice catch Robbie,

We add it before the release and removed it :)

So seeing is useful for user we will reintroduce it !!

Thank you Robbie for all your support!

Luciano

Original comment by luciano.broussal@gmail.com on 15 Feb 2007 at 12:52

GoogleCodeExporter commented 8 years ago

Original comment by marceloemanoel on 15 Feb 2007 at 1:43

GoogleCodeExporter commented 8 years ago
Fixed. GInternalFrameAdapter support available.

Original comment by luciano.broussal@gmail.com on 9 Apr 2007 at 6:52