theme: define proper :hover and :active states

- buttons didn't actually have these properly defined

fixes issue #523
This commit is contained in:
Jakub Steiner 2018-09-05 20:33:56 +02:00
parent 1bdb3f194d
commit 22531384f1
2 changed files with 7 additions and 6 deletions

View File

@ -48,17 +48,18 @@ stage {
padding: 4px 32px; padding: 4px 32px;
@include button(normal); @include button(normal);
&:focus { @include button(focus); } &:focus { @include button(focus); }
&:hover { @include button(hover); }
&:insensitive { @include button(insensitive); } &:insensitive { @include button(insensitive); }
&:active { @include button(active); } &:active { @include button(active); }
} }
.modal-dialog-linked-button { .modal-dialog-linked-button {
border-right-width: 1px; border-right-width: 1px;
@include button(normal); @include button(normal);
&:insensitive { @include button(insensitive); } &:insensitive { @include button(insensitive); }
&:active { @include button(active); }
&:focus { @include button(focus); } &:focus { @include button(focus); }
&:hover { @include button(hover); }
&:active { @include button(active); }
padding: 12px; padding: 12px;
&:first-child { &:first-child {

View File

@ -150,8 +150,8 @@
// //
// focused button // focused button
// //
$_bg: if($c!=$osd_bg_color, transparentize($c, 0.5), $_bg: if($c!=$osd_bg_color, transparentize($c, 0.3),
$osd_bg_color); lighten($osd_bg_color,3%));
color: $osd_fg_color; color: $osd_fg_color;
text-shadow: 0 1px black; text-shadow: 0 1px black;
@ -164,7 +164,7 @@
// active osd button // active osd button
// //
$_bg: if($c!=$osd_bg_color, transparentize($c, 0.3), $_bg: if($c!=$osd_bg_color, transparentize($c, 0.3),
lighten($osd_bg_color,10%)); lighten($osd_bg_color,3%));
color: white; color: white;
border-color: $osd_borders_color; border-color: $osd_borders_color;
@ -182,7 +182,7 @@
color: white; color: white;
border-color: $osd_borders_color; border-color: $osd_borders_color;
background-color: darken($_bg,5%); background-color: $selected_bg_color;
// This should be none, but it's creating some issues with borders, so to // This should be none, but it's creating some issues with borders, so to
// workaround it for now, use inset wich goes through a different code path. // workaround it for now, use inset wich goes through a different code path.
// see https://bugzilla.gnome.org/show_bug.cgi?id=752934 // see https://bugzilla.gnome.org/show_bug.cgi?id=752934