ZackandGitHub
ce74b5f614
* Refactor NotificationBadge to shared MVVM * Added snapshot images * Replace legacy NotificationBadge with adapter * Fix Eslint * Use regular NotificationBadge prop names * Remove legacy NotificationBadge selectors * Update TAC badge Playwright selectors * Rename NotificationBadgeAdapter to NotificationBadge Addresses review feedback: the component is the app-facing badge, not an adapter, so name it NotificationBadge. * Make NotificationBadgeView isClickable a required boolean Addresses review feedback: there is no semantic difference between `isClickable: false` and `isClickable: undefined` (both render the non-interactive variant), so the field should be a plain boolean. The view model always provides it. * Drop redundant comparison guards in NotificationBadge view model setters Addresses review feedback: Snapshot.merge already compares each field with Object.is and only emits when something actually changed, so the manual equality guards in each setter were redundant. Recompute the snapshot unconditionally and let merge dedupe. * Resolve notification badge labels inside NotificationBadgeView Addresses review feedback: static UI labels belong in the view, not the snapshot. The view now resolves the knock and unsent-message strings via useI18n, and the snapshot exposes a showUnsentTooltip boolean instead of a prebuilt tooltipLabel (and drops knockLabel entirely). Adds the room|knock_sent and notifications|message_didnt_send keys to the shared-components i18n catalogue so they resolve in tests and storybook. * Remove unused translation because of shared components movement * Fix typescript issue * Fix lint issue * Remove unused notification badge count setting * Pass notification badge class names explicitly
103 lines
2.9 KiB
Plaintext
103 lines
2.9 KiB
Plaintext
/*
|
|
* Copyright 2024 New Vector Ltd.
|
|
* Copyright 2024 The Matrix.org Foundation C.I.C.
|
|
*
|
|
* SPDX-License-Identifier: AGPL-3.0-only OR GPL-3.0-only OR LicenseRef-Element-Commercial
|
|
* Please see LICENSE files in the repository root for full details.
|
|
*/
|
|
|
|
.mx_ThreadsActivityCentre_container {
|
|
display: flex;
|
|
}
|
|
|
|
.mx_ThreadsActivityCentreButton {
|
|
/* !important override compound */
|
|
border-radius: 8px !important;
|
|
margin: 18px auto auto auto;
|
|
|
|
&.expanded {
|
|
/**
|
|
* override compound default background color when hovered
|
|
* should disappear when the space panel will be migrated to compound
|
|
*/
|
|
background-color: transparent !important;
|
|
|
|
/* align with settings icon */
|
|
margin-left: 21px;
|
|
margin-right: 8px;
|
|
width: 100%;
|
|
|
|
/**
|
|
* modify internal css of the compound component
|
|
* dirty but we need to add the `Threads` label into the indicator icon button
|
|
**/
|
|
& > div {
|
|
display: flex;
|
|
align-items: center;
|
|
|
|
&::before {
|
|
inline-size: 8px;
|
|
block-size: 8px;
|
|
left: 17px;
|
|
}
|
|
}
|
|
|
|
& .mx_ThreadsActivityCentreButton_Icon {
|
|
/* align with settings label */
|
|
margin-right: 14px;
|
|
/* required to set the icon width when into a flex container */
|
|
width: 24px;
|
|
flex-shrink: 0;
|
|
}
|
|
|
|
& .mx_ThreadsActivityCentreButton_Text {
|
|
/* !important override compound */
|
|
color: $secondary-content !important;
|
|
}
|
|
}
|
|
|
|
&:not(.expanded) {
|
|
&:hover,
|
|
&:hover .mx_ThreadsActivityCentreButton_Icon {
|
|
/* !important override compound */
|
|
background-color: $quaternary-content !important;
|
|
fill: $primary-content;
|
|
}
|
|
}
|
|
|
|
& .mx_ThreadsActivityCentreButton_Icon {
|
|
fill: $secondary-content;
|
|
}
|
|
}
|
|
|
|
.mx_ThreadsActivityCentre_rows {
|
|
overflow-y: scroll;
|
|
/* Let some space at the top and the bottom of the pop-up */
|
|
max-height: calc(100vh - 200px);
|
|
|
|
.mx_ThreadsActivityCentreRow {
|
|
height: 48px;
|
|
|
|
/* Make the label of the MenuItem stay on one line and truncate with ellipsis if needed */
|
|
& > span {
|
|
white-space: nowrap;
|
|
overflow: hidden;
|
|
text-overflow: ellipsis;
|
|
/* Arbitrary size, keep the TAC as the wanted width */
|
|
width: 202px;
|
|
}
|
|
|
|
/* Hide the notification badge on hover — compound's `nav-hint ~ *` rule would normally
|
|
* do this, but the app-web CSS layer overrides compound-web regardless of specificity. */
|
|
&:hover .mx_ThreadsActivityCentre_notificationBadge {
|
|
display: none;
|
|
}
|
|
}
|
|
}
|
|
|
|
.mx_ThreadsActivityCentre_emptyCaption {
|
|
padding-left: 16px;
|
|
padding-right: 16px;
|
|
font-size: 13px;
|
|
}
|