A few questions on Jewel 1. Are separators suppos...

# jewel

[Kirill Grouchnikov]

A few questions on Jewel

1. Are separators supposed to be super subtle? The two section headers (General and Propeller) have trailing separators, but they are almost invisible
2. What is the canonical way of getting a "secondary" or disabled text color? In Material there's onSurfaceVariant at the level of the theme. JewelTheme.checkboxStyle.colors.contentDisabled works, but it feels awkward to ask it from the checkbox style
3. Is there a color token for something along the lines of primary in Material? Some sort of a theme level accent color?

[seb]

Hey Kirill!
1. Is this the standalone flavor? The group dividers should be a bit more visible than that but the palettes have changed recently in Int UI and this might have slipped through
2. See <http://JewelTheme.globalColors.text.info|JewelTheme.globalColors.text.info> (going by memory, something like that)
3. Not really, Int UI as a design system didn't really have semantic colour tokens until ~a month ago and we haven't gotten there yet 🙂

[Kirill Grouchnikov]

Thanks, JewelTheme.globalColors.text.disabled is a good match for the second one.

It is the standalone flavor. I'm not passing an explicit color to Divider, so it should be picking up the color from the dividerStyle

[Kirill Grouchnikov]

Another question on the top-level configuration of the Jewel window. github.com/JetBrains/intellij-community/blob/…/Main.kt#… doesn't show decoratedWindow available for auto-completion on the latest version in Maven Central (0.39.1-262.9437.29).

Is Maven Central behind?

[seb]

> It is the standalone flavor. I'm not passing an explicit color to Divider, so it should be picking up the color from the dividerStyle
I am AFK right now, can you file a JEWEL ticket for this on youtrack? Will take a look tomorrow

[seb]

As for decoratedWindow, it's a separate dependency, have you set it up?

[seb]

Preliminary docs, still unreviewed, written by AI: jewel-ui.dev/docs/guides/decorated-windows
(not linked from anywhere since well, unreviewed slop, but it's the most I can help with rn 🙂)

[seb]

This part is correct at least: github.com/JetBrains/intellij-community/…/jewel#…

[Kirill Grouchnikov]

youtrack.jetbrains.com/issue/JEWEL-1416/Default-divider-color-in-standalone-version-is-too-subtle

SG for the decorated window, I will look it up a bit later today

[seb]

We changed how the decorated window deps are structured and IIRC now one is essentially just an empty shell (IIRC the int-ui-standalone version is now only a POM bringing in the dependency or something)

[Kirill Grouchnikov]

OK, now in JBR's dark mode.

One thing that I would consider investigating, @seb, is to set .background(JewelTheme.globalColors.panelBackground by default at the top level of the Layout inside the DecoratedWindow implementation. Took me a bit of time to figure out that it needs to be set on my content box to match the theme.

In the sample, you set it in github.com/JetBrains/intellij-community/blob/…/WelcomeView.kt#… but not in github.com/JetBrains/intellij-community/blob/…/Main.kt

[seb]

That's a good point, and we should document it. Bit resistant to doing it by default because there are technically two different valid background colours (e.g., in light theme, the grey and the white) that the theme palette works onto...