Skip to content

Use Split Button on item details screen#127

Merged
formatBCE merged 2 commits intomusic-assistant:mainfrom
seadowg:item-ui-polish
Feb 26, 2026
Merged

Use Split Button on item details screen#127
formatBCE merged 2 commits intomusic-assistant:mainfrom
seadowg:item-ui-polish

Conversation

@seadowg
Copy link
Copy Markdown
Collaborator

@seadowg seadowg commented Feb 26, 2026

Happy to find out that there was a multiplatform implementation - we just needed to upgrade!

Screenshot_20260226_101957

implementation(compose.ui)
implementation(compose.components.resources)
implementation(compose.components.uiToolingPreview)
implementation(libs.material)
Copy link
Copy Markdown
Collaborator Author

@seadowg seadowg Feb 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So it seems like the Compose Multiplatform library setup has changed since this app was kicked off. They know ship separate artifacts instead of attaching the libraries to the multiplatform plugin. This means the compose.* dependencies are currently deprecated, but still working. It's probably worth tidying these all up in a separate PR.

Comment thread gradle/libs.versions.toml
androidx-test-junit = "1.3.0"

compose-multiplatform = "1.9.3"
compose-multiplatform = "1.10.1"
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I needed to upgrade for the SplitButtonLayout. It's introduced some deprecations in some places (as well as for some dependency declarations as discussed in another comment), but lint/test is happy and everything builds fine.

@seadowg seadowg marked this pull request as ready for review February 26, 2026 10:21
Copy link
Copy Markdown
Collaborator

@formatBCE formatBCE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OMG this looks awesome.

@formatBCE formatBCE merged commit 9d3ce61 into music-assistant:main Feb 26, 2026
1 check passed
@seadowg seadowg deleted the item-ui-polish branch February 27, 2026 09:55
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.

2 participants