Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Fix avatar position of hidden event on ThreadView #8592

Merged
merged 3 commits into from
May 14, 2022
Merged

Fix avatar position of hidden event on ThreadView #8592

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ef38b9f
Fix avatar position of hidden events
luixxiul May 14, 2022
a1facfe
Override left padding of info event tile's line
luixxiul May 14, 2022
e8bb41d
Merge branch 'develop' into ThreadView_EventTile_info
luixxiul May 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 13 additions & 2 deletions res/css/views/right_panel/_ThreadPanel.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,6 +239,15 @@ limitations under the License.

&.mx_ThreadView .mx_EventTile {
// handling for hidden events (e.g reactions) in the thread view

&:not([data-layout=bubble]) {
&:hover.mx_EventTile_verified.mx_EventTile_info .mx_EventTile_line,
&:hover.mx_EventTile_unverified.mx_EventTile_info .mx_EventTile_line,
&:hover.mx_EventTile_unknown.mx_EventTile_info .mx_EventTile_line {
padding-inline-start: 0; // Override
}
}

&.mx_EventTile_info {
padding-top: 0;

Expand All @@ -263,20 +272,22 @@ limitations under the License.
}

.mx_EventTile_avatar {
top: 0; // Align with hidden event content
left: calc($MessageTimestamp_width + 14px - 4px); // 14px: avatar width, 4px: align with text
z-index: 9; // position above the hover styling
}
}

&[data-layout=bubble] {
.mx_EventTile_avatar {
left: 30px;
inset-inline-start: 0;
}
}

.mx_EventTile_avatar {
position: absolute;
top: 1.5px; // Align with hidden event content
margin-top: 0;
margin-bottom: 0;
width: 14px; // avatar img size
height: 14px; // avatar img size
}
Expand Down