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

Adjusted structure of material widgets. #2941

Merged
merged 1 commit into from Jun 20, 2023
Merged

Conversation

FloVanGH
Copy link
Member

@FloVanGH FloVanGH commented Jun 20, 2023

  • Adjust structure of material widgets to match new structure of fluent widgets
  • Fixes min-width and min-height in some widgets (according to the fluent variant)
  • Replaces Paths with Images so that all material widgets are working also with swr

@@ -82,3 +82,7 @@ License: Apache-2.0
Files: internal/compiler/widgets/fluent-base/_*.svg
Copyright: Copyright © Microsoft Corporation <https://github.com/microsoft/fluentui-system-icons/blob/main/LICENSE>
License: MIT

Files: internal/compiler/widgets/material-base/_*.svg
Copyright: Material Icons <https://github.com/material-icons/material-icons/blob/master/LICENSE>
Copy link
Member

Choose a reason for hiding this comment

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

small details: but i wonder if it would not be better to add these kind of copyright in the svg themselves.
(since these images are uploaded in the crate package for the compiler where .reuse is not available)
But it doesn't matter much anyway.

@FloVanGH FloVanGH merged commit 18d5680 into master Jun 20, 2023
26 checks passed
@FloVanGH FloVanGH deleted the florian/material-structure branch June 23, 2023 12:06
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

2 participants