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

Improve style rules for thread summary #8868

Merged
merged 20 commits into from
Jul 4, 2022
Merged

Improve style rules for thread summary #8868

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
feba803
Use mixin ThreadSummaryIcon
luixxiul Apr 28, 2022
2a1bd18
Tidy mx_ThreadSummary
luixxiul Jun 18, 2022
d9d8c9f
Move style blocks from _EventTile.scss to _ThreadSummary.scss
luixxiul Jun 18, 2022
7b93164
Merge mx_ThreadSummaryIcon::before
luixxiul Jun 18, 2022
7426736
From threads amount to replies amount
luixxiul Jun 18, 2022
9b7a40a
Remove obsolete declaration and class names
luixxiul Jun 18, 2022
2a12087
Move mx_ThreadPanel_replies::before from _ThreadSummary to _EventTile…
luixxiul Jun 18, 2022
d065cd3
Rename mx_ThreadSummaryIcon to mx_ThreadSummary_icon
luixxiul Jun 18, 2022
7ccc3c9
Use variables and remove obsolete one
luixxiul Jun 18, 2022
91bd4cd
Merge style rules, renaming a variable
luixxiul Jun 18, 2022
55fdbe9
Include mx_MessagePanel_narrow in mx_ThreadSummary
luixxiul Jun 19, 2022
c33b72f
Remove a redundant declaration
luixxiul Jun 19, 2022
fc43d9c
Use a variable
luixxiul Jun 19, 2022
2e06664
Include mx_ThreadSummary_sender and mx_ThreadSummary_content in mx_Th…
luixxiul Jun 19, 2022
0cf35db
Remove a variable used only once
luixxiul Jun 19, 2022
8455c50
Ensure the same line-height is applied
luixxiul Jun 19, 2022
3256d48
Remove !important
luixxiul Jun 25, 2022
36827ba
Merge remote-tracking branch 'upstream/develop' into ThreadSummary
luixxiul Jul 1, 2022
8decfa7
Merge branch 'develop' into ThreadSummary
luixxiul Jul 1, 2022
d7267a5
Merge branch 'develop' into ThreadSummary
luixxiul Jul 4, 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
6 changes: 5 additions & 1 deletion res/css/views/rooms/_EventTile.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,11 @@ limitations under the License.
$left-gutter: 64px;

.mx_EventTile {
--EventTile-gutter-start: $left-gutter;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was removed in favor of --EventTile_group_line-spacing-inline-start.

--EventTile_content-margin-inline-end: 34px; // TODO: Use a spacing variable
--EventTile_group_line-spacing-block-start: 1px;
--EventTile_group_line-spacing-block-end: 3px;
--EventTile_group_line-spacing-inline-start: $left-gutter;
--EventTile_group_line-line-height: $font-22px;
--EventTile_ThreadSummary-line-height: calc(2 * $font-12px);

flex-shrink: 0;
Expand Down
2 changes: 1 addition & 1 deletion res/css/views/rooms/_ThreadSummary.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ limitations under the License.

.mx_ThreadSummary {
min-width: 267px;
max-width: min(calc(100% - var(--EventTile-gutter-start)), 600px); // leave space on both left & right gutters
max-width: min(calc(100% - var(--EventTile_group_line-spacing-inline-start)), 600px); // leave space on both left & right gutters
width: fit-content;
height: 40px;
position: relative;
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.