fix(styles): refactor Notifications component [ci visual #5616
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue
Closes #5598
Description
The Notifications component has been updated to latest design; includes Breaking changes to accommodate the design changes and the a11y requirements. Removed the BTP examples and code, refactored and cleaned-up the code.
BREAKING CHANGES:
Before:
After:
fd-notification--mobile
modifier is removed. The responsiveness is handled automatically with media queries.fd-notification--in-dialog
modifier is removed.fd-notification--padding
modifier is removed.fd-notification--group
modifier is removed. Grouping is handled with different markup.fd-notification--btp
modifier is removed, BTP component is marked as legacy in the design.fdb-notification__group-header-title
is removed, BTP component is marked as legacy in the design.fd-notification__unread-indicator
is removed, BTP component is marked as legacy in the design.fd-notification__limit
is removed, BTP component is marked as legacy in the design.fd-notification__group-header--collapsed
modifier is removed.Before:
After: