Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refine 'addComposeTab' to include closeable option #118

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

lamba92
Copy link
Collaborator

@lamba92 lamba92 commented Sep 19, 2023

Introduced 'isCloseable' parameter in 'addComposeTab' function to enable flexibility in controlling behavior of tabs. This option allows programmatically defining whether a tab can be closed or not. Default value set to false to maintain existing function calls.

Introduced 'isCloseable' parameter in 'addComposeTab' function to enable flexibility in controlling behavior of tabs. This option allows programmatically defining whether a tab can be closed or not. Default value set to false to maintain existing function calls.
@lamba92 lamba92 requested review from rock3r and fscarponi September 19, 2023 15:22
@fscarponi fscarponi merged commit 33c2750 into main Sep 19, 2023
6 checks passed
@fscarponi fscarponi deleted the lamberto.basti/closeabletool-window-param branch September 19, 2023 15:45
Copy link
Collaborator

@rock3r rock3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOOO STOP WHAT THE HELL

.apply { setContent(content) }
.also { contentManager.addContent(contentManager.factory.createContent(it, tabDisplayName, isLockable)) }
) {
System.setProperty("compose.swing.render.on.graphics", "true")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WTF why is this here

rock3r added a commit that referenced this pull request Sep 19, 2023
@rock3r
Copy link
Collaborator

rock3r commented Sep 19, 2023

Sigh I guess I have to fix this on main now 🙄

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.

3 participants