New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
COMPASS-392: Connect window sidebar styles #713
Conversation
@@ -63,7 +63,7 @@ | |||
|
|||
.list-group-item { | |||
a { | |||
padding: 2px 0px 6px 28px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can I get a screenshot? I left less padding on top since the majority of the visual weight lies in the hostname which should feel more vertically centered than the timestamp.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
localhost:27017 doesn't have any descenders so it feels more balanced. I would take it down to 3px or 4px padding on top max.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🎊
9e00a9b
to
39cdd09
Compare
39cdd09
to
adf83c8
Compare
NOTE: I did not clean any of these styles up.
@Sean-Oh