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

Mdc migration chip, button, menu, table, checkbox and list #3706

Merged
merged 83 commits into from
Jul 11, 2024

Conversation

bastianjoel
Copy link
Member

@bastianjoel bastianjoel commented May 28, 2024

resolves #2934
closes #3694
closes #3696
closes #3703

@bastianjoel bastianjoel self-assigned this May 28, 2024
@bastianjoel
Copy link
Member Author

We should maybe replace the chips with an custom component.
@Elblinator maybe you could have a look on that?

@Elblinator Elblinator assigned rrenkert and Elblinator and unassigned MSoeb Jul 9, 2024
@bastianjoel
Copy link
Member Author

bastianjoel commented Jul 10, 2024

Checkboxes have different inside colours

I did this on purpose because I thought the contrast is quite bad and did not notice that it was black previously. The presence button uses a transparent icon and I cannot change it easily there. Should I change it back to black or is it okay as it is now?

@bastianjoel bastianjoel force-pushed the 2934-rest branch 2 times, most recently from 17a5262 to e9b6412 Compare July 10, 2024 12:47
@bastianjoel bastianjoel merged commit 44b7533 into OpenSlides:main Jul 11, 2024
3 checks passed
@bastianjoel bastianjoel deleted the 2934-rest branch July 11, 2024 11:03
Copy link
Member

@reiterl reiterl left a comment

Choose a reason for hiding this comment

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

I am not sure, if it is an issue, but I found some location in the source code, where:
class, ngClass and [class] are mixed. Else look good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Angular material mdc migration
5 participants