Skip to content

Divide required and optional plugin settings - #20

Open
matteomekhail wants to merge 24 commits into
Wip-Sama:devfrom
matteomekhail:codex/issue-12-divide-plugin-settings
Open

Divide required and optional plugin settings#20
matteomekhail wants to merge 24 commits into
Wip-Sama:devfrom
matteomekhail:codex/issue-12-divide-plugin-settings

Conversation

@matteomekhail

Copy link
Copy Markdown

Summary

  • split custom plugin settings into Required and Optional groups
  • show a count for each group while preserving search and sidebar navigation
  • localize the new group labels in English and Italian

Test

GRADLE_USER_HOME=/private/tmp/orlando-gradle bash gradlew :composeApp:jvmTest --tests org.wip.plugintoolkit.features.plugin.ui.PluginSettingsLayoutRegressionTest

Closes #12

…ue-12-divide-plugin-settings

# Conflicts:
#	composeApp/src/commonMain/kotlin/org/wip/plugintoolkit/features/plugin/ui/PluginSettingsContent.kt
…ue-12-divide-plugin-settings

# Conflicts:
#	composeApp/src/commonMain/kotlin/org/wip/plugintoolkit/features/plugin/logic/PluginLifecycleManager.kt
…sue-11-custom-setting-defaults"

This reverts commit ce2f1a8, reversing
changes made to 2f15ae1.
…dex/issue-12-divide-plugin-settings"

This reverts commit db3dfd2, reversing
changes made to c6cb1eb.
…setting-defaults

# Conflicts:
#	composeApp/src/commonMain/kotlin/org/wip/plugintoolkit/features/plugin/ui/DirectExecutionSidebar.kt
#	composeApp/src/commonMain/kotlin/org/wip/plugintoolkit/features/plugin/ui/PluginContent.kt
…e-12-divide-plugin-settings

# Conflicts:
#	composeApp/src/commonMain/composeResources/values/strings.xml
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.

1 participant