Update plain notification bell buttons in masthead to use notification badge

patternfly / patternfly-react

A set of React components for the PatternFly project.

https://react-staging.patternfly.org/

MIT License

800 stars 355 forks source link

Update plain notification bell buttons in masthead to use notification badge #10625

Open mcoker opened 5 months ago

mcoker commented 5 months ago

There are some plain buttons with a bell icon that look like notification badges that should use the <NotificationBadge> component. For example in the dashboardwrapper

https://github.com/patternfly/patternfly-react/blob/c9886cacdd291d0ead56bd43ff15a49880a7eb3a/packages/react-core/src/demos/DashboardHeader.tsx#L106-L115

github-actions[bot] commented 3 months ago

This issue has been automatically marked as stale because it has not had recent activity.

tlabaj commented 1 week ago

Looks like it is Page demos, Nave demos, Masthead demos and Notification drawer demos as well.