Skip to content

Commit

Permalink
time pills: Convert most uses of rem to px.
Browse files Browse the repository at this point in the history
This is a pure refactor, in the default case where the root font size
is 16px; we convert the values exactly for that case, and in the next
commit we'll adjust them to values that align nicely to our grid.

When the font size is expanded, this makes a welcome change:
previously the larger font would cause the pills' padding and inset
margin to grow, encroaching on the content precisely when it's likely
to be crowded already.  Now it doesn't.
  • Loading branch information
gnprice committed Jun 11, 2019
1 parent e16094a commit 3dc7fee
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/webview/css/css.js
Original file line number Diff line number Diff line change
Expand Up @@ -115,12 +115,12 @@ hr {
}
.timestamp {
position: absolute;
right: 0.275rem;
right: 4.4px;
transform: translateX(100%);
transition-property: right, transform;
transition-duration: 0.2s;
width: 4.3rem;
padding: 0.112rem 0;
padding: 1.79px 0;
font-size: 0.9rem;
text-align: center;
white-space: nowrap;
Expand All @@ -130,7 +130,7 @@ hr {
box-shadow: -1px 1px 2px 0 hsla(0, 0%, 0%, 0.3), -2px 2px 4px 0 hsla(0, 0%, 0%, 0.3);
}
.timestamp.show {
right: 0.705rem;
right: 11.28px;
transform: none;
}
.message p + p {
Expand Down

0 comments on commit 3dc7fee

Please sign in to comment.