Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change DefaultSkin[Skin] to DefaultSkin { type Default } #2

Merged
merged 1 commit into from
Jun 11, 2023

Conversation

JD557
Copy link
Owner

@JD557 JD557 commented Jun 11, 2023

Changes the DefaultSkin shape and adds explicit types so that the default skins can be modified with copy.

Previously the type inference was not strict enough and would infer the super type (e.g. WindowSkin.lightDefault was inferred as WindowSkin instead of WindowSkin.Default).

@JD557 JD557 added this to the 0.1.1 milestone Jun 11, 2023
@JD557 JD557 merged commit bdedb12 into master Jun 11, 2023
8 checks passed
@JD557 JD557 deleted the fix-default-skins branch June 11, 2023 15:45
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.

None yet

1 participant