From 78fcfeb1079d8fc6f8f25d01ea66497f83cfa200 Mon Sep 17 00:00:00 2001 From: Sebastiano Poggi Date: Mon, 23 Oct 2023 10:42:59 +0100 Subject: [PATCH] Fix disabled default buttons in old UI We were reading the wrong values... took the opportunity to use JBUI as much as possible for buttons, to be on the safe side. --- .../jewel/bridge/theme/IntUiBridge.kt | 36 ++++++++++++------- 1 file changed, 23 insertions(+), 13 deletions(-) diff --git a/ide-laf-bridge/src/main/kotlin/org/jetbrains/jewel/bridge/theme/IntUiBridge.kt b/ide-laf-bridge/src/main/kotlin/org/jetbrains/jewel/bridge/theme/IntUiBridge.kt index 7a46b1c14c..501d8fdaeb 100644 --- a/ide-laf-bridge/src/main/kotlin/org/jetbrains/jewel/bridge/theme/IntUiBridge.kt +++ b/ide-laf-bridge/src/main/kotlin/org/jetbrains/jewel/bridge/theme/IntUiBridge.kt @@ -196,14 +196,18 @@ internal fun createBridgeComponentStyling( private fun readDefaultButtonStyle(): ButtonStyle { val normalBackground = - retrieveColorsOrUnspecified( - "Button.default.startBackground", - "Button.default.endBackground", + listOf( + JBUI.CurrentTheme.Button.defaultButtonColorStart().toComposeColor(), + JBUI.CurrentTheme.Button.defaultButtonColorEnd().toComposeColor(), ).createVerticalBrush() + val normalContent = retrieveColorOrUnspecified("Button.default.foreground") + val normalBorder = - retrieveColorsOrUnspecified("Button.default.startBorderColor", "Button.default.endBorderColor") - .createVerticalBrush() + listOf( + JBUI.CurrentTheme.Button.buttonOutlineColorStart(true).toComposeColor(), + JBUI.CurrentTheme.Button.buttonOutlineColorEnd(true).toComposeColor(), + ).createVerticalBrush() val colors = ButtonColors( background = normalBackground, @@ -212,12 +216,12 @@ private fun readDefaultButtonStyle(): ButtonStyle { backgroundPressed = normalBackground, backgroundHovered = normalBackground, content = normalContent, - contentDisabled = retrieveColorOrUnspecified("Button.default.disabledText"), + contentDisabled = retrieveColorOrUnspecified("Button.disabledText"), contentFocused = normalContent, contentPressed = normalContent, contentHovered = normalContent, border = normalBorder, - borderDisabled = SolidColor(retrieveColorOrUnspecified("Button.default.disabledBorderColor")), + borderDisabled = SolidColor(JBUI.CurrentTheme.Button.disabledOutlineColor().toComposeColor()), borderFocused = SolidColor(retrieveColorOrUnspecified("Button.default.focusedBorderColor")), borderPressed = normalBorder, borderHovered = normalBorder, @@ -236,12 +240,18 @@ private fun readDefaultButtonStyle(): ButtonStyle { private fun readOutlinedButtonStyle(): ButtonStyle { val normalBackground = - retrieveColorsOrUnspecified("Button.startBackground", "Button.endBackground") - .createVerticalBrush() + listOf( + JBUI.CurrentTheme.Button.buttonColorStart().toComposeColor(), + JBUI.CurrentTheme.Button.buttonColorEnd().toComposeColor(), + ).createVerticalBrush() + val normalContent = retrieveColorOrUnspecified("Button.foreground") + val normalBorder = - retrieveColorsOrUnspecified("Button.startBorderColor", "Button.endBorderColor") - .createVerticalBrush() + listOf( + JBUI.CurrentTheme.Button.buttonOutlineColorStart(false).toComposeColor(), + JBUI.CurrentTheme.Button.buttonOutlineColorEnd(false).toComposeColor(), + ).createVerticalBrush() val colors = ButtonColors( background = normalBackground, @@ -255,8 +265,8 @@ private fun readOutlinedButtonStyle(): ButtonStyle { contentPressed = normalContent, contentHovered = normalContent, border = normalBorder, - borderDisabled = SolidColor(retrieveColorOrUnspecified("Button.disabledBorderColor")), - borderFocused = SolidColor(retrieveColorOrUnspecified("Button.focusedBorderColor")), + borderDisabled = SolidColor(JBUI.CurrentTheme.Button.disabledOutlineColor().toComposeColor()), + borderFocused = SolidColor(JBUI.CurrentTheme.Button.focusBorderColor(false).toComposeColor()), borderPressed = normalBorder, borderHovered = normalBorder, )