Skip to content

fix: call setVisible(true) on SessionGUI instance to display the panel#95

Open
AmlanDalai wants to merge 3 commits intosumit3203:masterfrom
AmlanDalai:fix-sessiongui-bug
Open

fix: call setVisible(true) on SessionGUI instance to display the panel#95
AmlanDalai wants to merge 3 commits intosumit3203:masterfrom
AmlanDalai:fix-sessiongui-bug

Conversation

@AmlanDalai
Copy link
Copy Markdown
Contributor

Bug Fix

Problem

In GuiPanel.java, when the "View Sessions" button was clicked,
a new SessionGUI instance was created but never made visible:

// Before (buggy)
new SessionGUI(projectManager); // panel never shown to user

Fix

Store the instance and call setVisible(true) so the panel
actually appears when the button is clicked:

// After (fixed)
SessionGUI sessionGUI = new SessionGUI(projectManager);
sessionGUI.setVisible(true);

Impact

Users clicking "View Sessions" button would see nothing happen.
This fix makes the Sessions panel actually appear as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant