Patrick
Just saw the commit. Great!!!
Here is another one.
The change is in module Gui5250Frame.java method name removeSessionView.
This fixes a problem with the closing the last session tab on the frame by
clicking on the X close icon. It causes an error and does not shut down all
the sessions because the session counts are off. This also was fix for the
Last Session View as well because the counts are not zero then the new session
view was not being saved at the end.
------------------------------------------------------------------------------
public void removeSessionView(SessionGUI targetSession) {
if (hideTabBar && sessionPane.getTabCount() == 0) {
for (int x=0; x < getContentPane().getComponentCount(); x++) {
if (getContentPane().getComponent(x) instanceof SessionGUI) {
getContentPane().remove(x);
}
}
}
else {
int index = sessionPane.indexOfComponent(targetSession);
log.info("session found and closing down " + index);
targetSession.removeSessionListener(this);
targetSession.removeSessionJumpListener(this);
sessionPane.remove(index);
// get the tab count to be used later
int tabs = sessionPane.getTabCount();
// if the index removed is the same as the number of tabs
// we need to decrement the index offset or we will get
// an error because we went over the tab limit which starts
// at zero offset.
if (tabs == index)
index--;
if (tabs > 0 && index < tabs) {
sessionPane.setSelectedIndex(index);
sessionPane.setForegroundAt(index,Color.blue);
sessionPane.setIconAt(focused,index);
((SessionGUI)sessionPane.getComponentAt(index)).requestFocus();
}
else {
if (tabs > 0) {
sessionPane.setSelectedIndex(0);
sessionPane.setForegroundAt(0,Color.blue);
sessionPane.setIconAt(focused,0);
((SessionGUI)sessionPane.getComponentAt(0)).requestFocus();
}
}
}
}
----------------------------------------------------------------------
-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Tn5250j-general mailing list
Tn5250j-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tn5250j-general
