Skip to content

Commit

Permalink
Make spacing at top/bottom of last seen indicator consistent
Browse files Browse the repository at this point in the history
Sadly, that spacing now needs to be bigger because scrollIntoView
doesn't honor margins which overlap each other to eliminate this kind of
'too much spacing' kind of problem.

FREEBIE
  • Loading branch information
scottnonnenberg committed Jun 9, 2017
1 parent ed56d8c commit 13fb40f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
9 changes: 6 additions & 3 deletions stylesheets/_conversation.scss
Expand Up @@ -696,9 +696,12 @@ li.entry .error-icon-container {
}

.message-list .last-seen-indicator-view {
// We need this extra layer of DOM nodes because scrollIntoView() doesn't honor margins
padding-bottom: 2em;
padding-top: 2em;
// This padding is large so we clear the avatar circle extending into the conversation
// window.scrollIntoView() doesn't honor margins, so we're using padding
// padding-top is less to account for the 10px margin at the bottom of messages
padding-top: 25px;
padding-bottom: 35px;


.bar {
display: flex;
Expand Down
4 changes: 2 additions & 2 deletions stylesheets/manifest.css

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 13fb40f

Please sign in to comment.