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

Feature: Added Notification Type, Type Visibility #13

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

JannesStroehlein
Copy link

I added notification types and a property to opt out of showing the notification type. The generic theme now displays the notification type with a accented icon.

New Notification Type Icons

image

Things I have changed / added

  • added interface definition (INotificationMessage)
  • added properties and styled properties to control (NotificationMessage)
  • added builder and linq builder methods to set the type and type visibility
  • added property changed handler to update the pseudoclasses of the control when the type or type visibility are changed
  • removed redundant class that was added by the constructor of NotificationMessage, because there was a more elegant way of selecting the textblocks
  • added icons licensed under the permissive "Unlicense" license to the theme
  • updated sample project to show the types in actions

Every code that I added, which is exposed by the API, has the proper code docs and most of the internal code I changed is also documented.

JannesStroehlein and others added 5 commits June 24, 2024 13:22
- added interface definition (INotificationMessage)
- added properties and styled properties to control (NotificationMessage)
- added builder and linq builder methods to set the type and type visibility
- added property changed handler to update the pseudoclasses of the control when the type or type visibility are changed
- removed redundant class that was added by the constructor of NotificationMessage, because there was a more elegant way of selecting the textblocks
- started work on style changes
- updated the sample to show the icons
…tifications that have the close button visibility set to true
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.

1 participant