Skip to content

Commit 07c7ce3

Browse files
authored
UX: minor style adjustments for rewind on mobile (#36633)
Some minor adjustments for small screen based on user feedback Emoji misalignment seen as a mistake rather than intentional, don't care enough to keep it! Before: <img width="568" height="304" alt="image" src="https://github.com/user-attachments/assets/37c65a4c-b45f-4af4-9b57-9ac789dbab6f" /> After: <img width="568" height="248" alt="image" src="https://github.com/user-attachments/assets/6886fb58-be97-49fc-aff8-8c3ba5849a80" /> On smaller screens, the header tab style looks like a mistake rather than intentional when the name is this long... so I just switch the style for smaller screens to avoid it Before: <img width="1070" height="1040" alt="image" src="https://github.com/user-attachments/assets/32b3ac27-2ada-477a-9be3-37c8dc47815d" /> After: <img width="1068" height="1060" alt="image" src="https://github.com/user-attachments/assets/e4db8297-7d41-47e7-8f6f-804fdc03657c" />
1 parent 16abc75 commit 07c7ce3

File tree

2 files changed

+15
-20
lines changed

2 files changed

+15
-20
lines changed

plugins/discourse-rewind/assets/stylesheets/common/post-received-reactions.scss

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -121,30 +121,11 @@
121121
@media screen and (width <= 475px) {
122122
.scale {
123123
width: min-content;
124-
}
125-
126-
.scale:nth-child(1) {
127-
padding: 0.15em;
128-
display: none;
129-
}
130-
131-
.scale:nth-child(2) {
132-
padding: 0.2em;
133-
margin-top: 10px;
134-
}
135-
136-
.scale:nth-child(3) {
137-
padding: 0.2em;
138-
margin-top: 20px;
139-
}
140-
141-
.scale:nth-child(4) {
142124
padding: 0.2em;
143125
}
144126

145127
.scale:nth-child(5) {
146-
padding: 0.2em;
147-
margin-top: 20px;
128+
display: none;
148129
}
149130
}
150131
}

plugins/discourse-rewind/assets/stylesheets/common/post-used-reactions.scss

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,20 @@
1111
width: 100%;
1212
}
1313

14+
@media screen and (width <= 600px) {
15+
.rewind-report-title {
16+
width: 100%;
17+
box-sizing: border-box;
18+
border-radius: var(--rewind-border-radius) var(--rewind-border-radius) 0 0;
19+
padding-top: var(--space-4);
20+
}
21+
22+
.rewind-card {
23+
border-radius: 0 0 var(--rewind-border-radius) var(--rewind-border-radius);
24+
padding-top: 0;
25+
}
26+
}
27+
1428
.rewind-reactions-row {
1529
display: grid;
1630
grid-template-columns: 25px 50px 1fr;

0 commit comments

Comments
 (0)