Skip to content

Commit 434cd01

Browse files
authored
Merge pull request #650 from doki-theme/usabilityStuff
Usability Stuff
2 parents b606c1e + 32e0f0d commit 434cd01

7 files changed

Lines changed: 41 additions & 5 deletions

File tree

buildSrc/assets/templates/base.laf.template.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@
4141
"StatusBar.background": "baseBackground",
4242
"SearchMatch.foreground": "baseBackground",
4343
"Popup.background": "baseBackground",
44+
"Plugins.Button.installBackground": "baseBackground",
45+
4446
"UIDesigner.Canvas.background": "textEditorBackground",
4547
"Content.background": "textEditorBackground",
4648
"Table.background": "textEditorBackground",

buildSrc/assets/templates/light.laf.template.json

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,17 @@
77
"Notification.ToolWindow.informativeForeground": "#33412e",
88
"Notification.ToolWindow.warningForeground": "#5a5221",
99
"Notification.ToolWindow.errorForeground": "#704745"
10+
},
11+
"icons": {
12+
"ColorPalette": {
13+
"Checkbox.Foreground.Selected": "baseBackground",
14+
"Checkbox.Background.Selected": "accentColor",
15+
"Checkbox.Border.Selected": "accentColor",
16+
"Checkbox.Focus.Thin.Default": "baseBackground",
17+
"Checkbox.Focus.Thin.Selected": "baseBackground",
18+
"Checkbox.Focus.Wide": "accentColor",
19+
"Checkbox.Foreground.Disabled": "baseBackground",
20+
"Checkbox.Background.Disabled": "disabledColor"
21+
}
1022
}
1123
}

buildSrc/assets/themes/nekoPara/shigure/light/shigure.light.jetbrains.definition.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,8 @@
55
"name": "Doki Light v2,Doki Dim Light Folding"
66
},
77
"overrides": {},
8-
"ui": {}
8+
"ui": {
9+
"Plugins.lightSelectionBackground": "secondaryBackground",
10+
"Plugins.hoverBackground": "secondaryBackground"
11+
}
912
}

buildSrc/src/main/kotlin/BuildThemes.kt

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ open class BuildThemes : DefaultTask() {
264264
constructableLookAndFeel,
265265
initialParentTemplateName,
266266
),
267-
icons = getIcons(resolvedNamedColors, constructableLookAndFeel),
267+
icons = getIcons(resolvedNamedColors, constructableLookAndFeel, initialParentTemplateName),
268268
meta = masterThemeDefinition.meta ?: Collections.emptyMap()
269269
)
270270

@@ -425,12 +425,22 @@ open class BuildThemes : DefaultTask() {
425425
@Suppress("UNCHECKED_CAST")
426426
private fun getIcons(
427427
colors: Map<String, String>,
428-
constructableAsset: ConstructableAsset
428+
constructableAsset: ConstructableAsset,
429+
initialParentTemplateName: String
429430
): Map<String, Any> {
430431
val template = constructableAsset.definitions["base"]
431432
?: throw IllegalArgumentException("Expected constructable type '${constructableAsset.type}' to have type 'base'!")
433+
val parentTemplate = constructableAsset.definitions[initialParentTemplateName]
434+
?: throw IllegalArgumentException("Expected constructable type '${constructableAsset.type}' to have type '${initialParentTemplateName}'!")
435+
val parentIcons = template.icons?.toMutableMap() ?: mutableMapOf()
436+
val childIcons = parentTemplate.icons?.toMutableMap() ?: mutableMapOf()
437+
val combinedPalette = combineMaps(
438+
(parentIcons["ColorPalette"] ?: emptyMap<String, String>()) as StringDictionary<Any>,
439+
(childIcons["ColorPalette"] ?: emptyMap<String, String>()) as StringDictionary<Any>
440+
)
441+
parentIcons["ColorPalette"] = combinedPalette
432442
return resolveNamedColorsForMap(
433-
template.icons ?: emptyMap(),
443+
parentIcons,
434444
colors,
435445
)
436446
}

changelog/CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
Changelog
22
---
3+
# 88.4-1.9.4 [Usability Updates]
4+
5+
- Made Light Theme's checkboxes & radio buttons more usable.
6+
- Sigure's list item selection is readable now.
7+
- Updated Sagiri's syntax highlight to avoid looking like an error.
8+
39
# 88.3-1.9.3 [New UI Updates]
410

511
- Themed New UI Main Toolbar button pressed color.

changelog/RELEASE-NOTES.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,3 +7,6 @@
77
- Fixed the settings window in builds `231.6890.12`+
88
- Themed New UI header button hover & pressed color
99
- Updated New UI project tree spacing to mirror JetBrains' defaults.
10+
- Made Light Theme's checkboxes & radio buttons more usable.
11+
- Sigure's list item selection is readable now.
12+
- Updated Sagiri's syntax highlight to avoid looking like an error.

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# -> https://www.jetbrains.org/intellij/sdk/docs/reference_guide/intellij_artifacts.html
33

44
pluginGroup=io.unthrottled
5-
pluginVersion=88.3-1.9.3
5+
pluginVersion=88.4-1.9.4
66
pluginSinceBuild=222
77
pluginUntilBuild = 231.*
88

0 commit comments

Comments
 (0)