Skip to content

XamlX upgrade#6288

Open
sowelipililimute wants to merge 8 commits intospace-wizards:masterfrom
sowelipililimute:work/jblackquill/demo-xamlx
Open

XamlX upgrade#6288
sowelipililimute wants to merge 8 commits intospace-wizards:masterfrom
sowelipililimute:work/jblackquill/demo-xamlx

Conversation

@sowelipililimute
Copy link
Member

@sowelipililimute sowelipililimute commented Nov 7, 2025

Requires space-wizards/space-station-14#42615

This updates our XamlX to a newer version that lets us more easily support features such as templated controls and data binding without having to try and backport verifiable code fixes to our 5-year-old fork of XamlX.

Breaking Changes

Buttons have had their styling change to rely on inheritance from PushButton rather than a style class; this is because new behaviour wrt style classes is overriding instead of concatenative, and all uses of concatenation in content were for adding a single or more classes onto a button.

Style classes also have a space-delimited syntax for multiple classes instead of the older verbose list notation.

@sowelipililimute sowelipililimute force-pushed the work/jblackquill/demo-xamlx branch 2 times, most recently from 94f916f to e128d52 Compare January 10, 2026 05:36
@sowelipililimute
Copy link
Member Author

mostly working but xaml jit is broken on MethodAccessExceptions and i cannot figure out for the life of me what's going on there (needs content changes for how i reorganized the button styles)

@sowelipililimute sowelipililimute force-pushed the work/jblackquill/demo-xamlx branch from 0efc6dc to b71f8b9 Compare January 18, 2026 20:18
@sowelipililimute sowelipililimute force-pushed the work/jblackquill/demo-xamlx branch from b71f8b9 to fd8a120 Compare January 23, 2026 21:12
@sowelipililimute sowelipililimute marked this pull request as ready for review January 23, 2026 23:14
@sowelipililimute sowelipililimute added the S: Requires Content PR This PR breaks content and requires both to be merged together. label Jan 23, 2026
@Tayrtahn Tayrtahn linked an issue Jan 24, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S: Requires Content PR This PR breaks content and requires both to be merged together.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

XAML UI does not parse two style classes correctly

1 participant