Skip to content

Page - change notificationDrawer props to be more generic #8635

@mcoker

Description

@mcoker

re: #8541 (comment)

There are currently a props on the <Page> component (notificationDrawer and isNotificationDrawerExpanded) that attach a drawer to the page. AFAIK there is no requirement to use a notification drawer there - the props just update the page to support a drawer that appears over the main content area. Assuming we don't want to limit that use of a drawer in the page to notification drawer only, I wonder if it makes sense to rename those props to something more generic so it's clear it can be used for any sort of drawer content.

cc @mcarrano @mceledonia @tlabaj

Metadata

Metadata

Assignees

No one assigned

    Labels

    Breaking change 💥this change requires a major release and has API changes.

    Type

    Projects

    Status

    Backlog

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions