[Tn5250j-general] Another change sent in
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
RE: [Tn5250j-general] Another change sent in
Hi All, Fix committed to cvs, all compiles fine, but when using java 1.5.0_05 two warnings pop up... not sure how to fix them, cause removing the null fixes it but i guess that will give troubles on older jvm's like 1.4.2. Best Regards, Patrick has notified the sender that this message has been received.
Re: [Tn5250j-general] Another change sent in
Kenneth, That may be related to the fix I committed this summer. The session counts were off by one in that case as well (see my post form 7/14 if you are interested). Pete --- 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
