Skip to content
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

GH-1623: Update Hover State for Expanded View #408

Merged
merged 5 commits into from Jul 17, 2019
Merged
Changes from 1 commit
Commits
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

centered left caret
  • Loading branch information
tdtnguyen committed Jul 2, 2019
commit 92ee0ccf9401eb6655e5faae3e0e49bab9d28ddc
@@ -97,11 +97,13 @@
background-repeat: no-repeat;
background-image: url('../../app/images/panel/line.svg');
&:hover {
background-position: calc(50% - 1.75px);

This comment has been minimized.

@IAmThePan

IAmThePan Jul 8, 2019
Contributor

I think this looks better with:

background-position: calc(50% - 2px);
background-color: $ghosty-blue;
background-size: 7px 20px;
background-image: url('../../app/images/panel/caret-left.svg');
}
&.condensed:hover {
background-position: center;
This conversation was marked as resolved by IAmThePan

This comment has been minimized.

@IAmThePan

IAmThePan Jul 8, 2019
Contributor

I think this looks better with:

background-position: calc(50% - 1px);
background-image: url('../../app/images/panel/caret-right.svg');
}
}
ProTip! Use n and p to navigate between commits in a pull request.