Skip to content

Commit

Permalink
UX: fix some chat spacing issues (discourse#25)
Browse files Browse the repository at this point in the history
  • Loading branch information
awesomerobot authored Dec 21, 2023
1 parent 60c3313 commit fe3b319
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 5 deletions.
8 changes: 6 additions & 2 deletions common/common.scss
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@

#main-outlet {
box-sizing: border-box;
margin: 30px auto;
margin: 0 auto;
background: var(--secondary);
border-radius: 8px;
width: 100%;
Expand Down Expand Up @@ -415,6 +415,10 @@ body.has-full-page-chat {
}
}

.full-page-chat .c-navbar-container {
top: unset;
}

.chat-channel {
height: calc(100vh - (var(--header-offset) + 60px));
height: calc(100vh - (var(--header-offset) + 8em));
}
4 changes: 2 additions & 2 deletions desktop/desktop.scss
Original file line number Diff line number Diff line change
Expand Up @@ -268,11 +268,11 @@ nav.post-controls {
}

#main-outlet-wrapper {
margin-top: 2em;
.sidebar-wrapper {
margin: 30px 0;
background-color: var(--secondary);
border-radius: 8px;
top: calc(var(--header-offset) + 30px);
top: calc(var(--header-offset) + 2em);
height: calc(100vh - (var(--header-offset) + 60px));
@include box-shadow;
}
Expand Down
7 changes: 6 additions & 1 deletion mobile/mobile.scss
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ body {
}

#main-outlet {
margin: 1em auto;
padding: 10px;
width: 96%;
}
Expand Down Expand Up @@ -203,5 +204,9 @@ div.cooked img:not(.thumbnail) {
}

.chat-channel {
height: calc(100dvh - (var(--header-offset) + 60px));
height: calc(100vh - (var(--header-offset) + 11em));
}

.chat-scroll-to-bottom__arrow {
width: 33px; // minor centering adjustment on iOS
}

0 comments on commit fe3b319

Please sign in to comment.