* Move Room List to ListView - Also remove Space/Enter handing from keyboard navigation we can just leave the default behaviour of those keys and handle via onClick * Update rooms when the primary filter changes Otherwise when changing spaces, the filter does not reset until the next update to the RVS is made. * Fix stickyRow/scrollIntoView when switiching space or changing filters - Also remove the rest of space/enter keyboard handling use * Remove the rest of space/enter keyboard handling use * Remove useCombinedRef and add @radix-ui/react-compose-refs as we already depend on it - Also remove eact-virtualized dep * Update RoomList unit test * Update snapshots and unit tests * Fix e2e tests * Remove react-virtualized from tests * Fix e2e flake * Update more screenshots * Fix e2e test case where were should scroll to the top when the active room is no longer in the list * Move from gitpkg to package-patch * Update to latest react virtuoso release/api. Also pass spaceId to the room list and scroll the activeIndex into view when spaceId or primaryFilter change. * Use listbox/option roles to improve ScreenReader experience * Change onKeyDown e.stopPropogation to cover context menu * lint * Remove unneeded exposure of the listView ref Also move scrollIntoViewOnChange to useCallback * Update unit test and snapshot * Fix e2e tests and update screenshots * Fix unit test and snapshot * Update more unit tests * Fix keyboard shortcuts and e2e test * Fix another e2e and unit test * lint * Improve the naming for RoomResult and the documentation on it's fields meaning. Also update the login in RoomList to check for any change in filters, this is a bit more future proof for when we introduce multi select than using activePrimaryFilter. * Put back and fix landmark tests * Fix test import * Add comment regarding context object getting rendered. * onKeyDown should be optional * Use SpaceKey type on RoomResult * lint
81 lines
2.4 KiB
Plaintext
81 lines
2.4 KiB
Plaintext
/*
|
|
* Copyright 2025 New Vector Ltd.
|
|
*
|
|
* 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.
|
|
*/
|
|
|
|
/**
|
|
* The RoomListItemView has the following structure:
|
|
* button--------------------------------------------------|
|
|
* | <-12px-> container------------------------------------|
|
|
* | | room avatar <-8px-> content----------------|
|
|
* | | | room_name <- 20px ->|
|
|
* | | | --------------------| <-- border
|
|
* |-------------------------------------------------------|
|
|
*/
|
|
.mx_RoomListItemView {
|
|
/* Remove button default style */
|
|
background: unset;
|
|
border: none;
|
|
padding: 0;
|
|
text-align: unset;
|
|
|
|
cursor: pointer;
|
|
height: 48px;
|
|
width: 100%;
|
|
|
|
padding-left: var(--cpd-space-3x);
|
|
font: var(--cpd-font-body-md-regular);
|
|
|
|
.mx_RoomListItemView_content {
|
|
height: 100%;
|
|
flex: 1;
|
|
/* The border is only under the room name and the future hover menu */
|
|
border-bottom: var(--cpd-border-width-0-5) solid var(--cpd-color-bg-subtle-secondary);
|
|
box-sizing: border-box;
|
|
min-width: 0;
|
|
padding-right: var(--cpd-space-5x);
|
|
|
|
.mx_RoomListItemView_text {
|
|
min-width: 0;
|
|
}
|
|
|
|
.mx_RoomListItemView_roomName {
|
|
white-space: nowrap;
|
|
overflow: hidden;
|
|
text-overflow: ellipsis;
|
|
}
|
|
|
|
.mx_RoomListItemView_messagePreview {
|
|
font: var(--cpd-font-body-sm-regular);
|
|
color: var(--cpd-color-text-secondary);
|
|
white-space: nowrap;
|
|
overflow: hidden;
|
|
text-overflow: ellipsis;
|
|
}
|
|
}
|
|
}
|
|
|
|
.mx_RoomListItemView_hover {
|
|
background-color: var(--cpd-color-bg-action-secondary-hovered);
|
|
}
|
|
|
|
.mx_RoomListItemView_menu_open .mx_RoomListItemView_content {
|
|
/**
|
|
* The figma uses 16px padding (--cpd-space-4x) but due to https://github.com/element-hq/compound-web/issues/331
|
|
* the icon size of the menu is 18px instead of 20px with a different internal padding
|
|
* We need to use 18px to align the icon with the others icons
|
|
* 18px is not available in compound spacing
|
|
*/
|
|
padding-right: 18px;
|
|
}
|
|
|
|
.mx_RoomListItemView_selected {
|
|
background-color: var(--cpd-color-bg-action-secondary-pressed);
|
|
}
|
|
|
|
.mx_RoomListItemView_bold .mx_RoomListItemView_roomName {
|
|
font: var(--cpd-font-body-md-semibold);
|
|
}
|