-
Notifications
You must be signed in to change notification settings - Fork 36
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
Decorated dialog support #513
base: main
Are you sure you want to change the base?
Conversation
if (!JBR.isAvailable()) { | ||
error( | ||
"DecoratedWindow can only be used on JetBrainsRuntime(JBR) platform, " + | ||
"please check the document https://github.com/JetBrains/jewel#int-ui-standalone-theme", | ||
) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worth extracting into assertUsingJBR()
and reuse in DecoratedWindow, too?
} | ||
|
||
// Using undecorated window for linux | ||
val undecorated = DesktopPlatform.Linux == DesktopPlatform.Current |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit:
val undecorated = DesktopPlatform.Linux == DesktopPlatform.Current | |
val undecorated = DesktopPlatform.Current == DesktopPlatform.Linux |
import androidx.compose.runtime.getValue | ||
import androidx.compose.runtime.setValue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't look like these are used
@@ -1,6 +1,6 @@ | |||
[versions] | |||
commonmark = "0.22.0" | |||
composeDesktop = "1.7.0-dev1743" | |||
composeDesktop = "1.7.0-dev1750" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this needed? Otherwise I'd do it separatey so it shows in the release notes
@devkanro ping |
@devkanro ping again? |
Description
This PR provide DecoratedDialog support and upgrade the JBR api to 1.0.2
fixed #512