Skip to content

Commit

Permalink
#135 Notification panel scrolling issue (#1587)
Browse files Browse the repository at this point in the history
  • Loading branch information
nmgokhale authored Oct 17, 2023
1 parent 25cef13 commit e6445fd
Showing 1 changed file with 10 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,12 @@
* limitations under the License.
*/

$notification-panel-header-height: 35px;
$notification-panel-header-height: 33px;
$notification-panel-subtitle-height: 20px;
$notification-message-height: 48px;
$notification-left-border-color-width: 4px;
$notification-panel-button-container-height: 49px;
$notification-panel-header-container-height: $notification-panel-header-height + $notification-panel-subtitle-height + 32px;
$toolbar-button-height: 41px; // Allow one extra pixel for the toolbar border

// This section adds an ease-in ease-out effect for the notification-panel-container
Expand All @@ -37,7 +40,8 @@ $toolbar-button-height: 41px; // Allow one extra pixel for the toolbar border
.notification-panel {
position: absolute;
width: 320px;
height: fit-content;
height: 100%;
max-height: 635px;
right: 0;
top: $toolbar-button-height;
}
Expand Down Expand Up @@ -70,7 +74,7 @@ $toolbar-button-height: 41px; // Allow one extra pixel for the toolbar border

.notification-panel-button-container {
width: 100%;
height: 49px;
height: $notification-panel-button-container-height;
background-color: $ui-01;
display: flex;
justify-content: space-between;
Expand All @@ -82,11 +86,11 @@ $toolbar-button-height: 41px; // Allow one extra pixel for the toolbar border

.notification-panel-messages-container {
width: inherit;
max-height: calc(100% - #{$notification-panel-header-height});
height: fit-content;
max-height: calc(100% - #{$notification-panel-header-height} - #{$notification-panel-subtitle-height});
height: 100%;
}
.notification-panel-messages {
max-height: 500px;
max-height: calc(100% - #{$notification-panel-header-container-height} - #{$notification-panel-button-container-height});
height: fit-content;
overflow-y: auto;
line-height: normal;
Expand Down

0 comments on commit e6445fd

Please sign in to comment.