Browse Source

Fix: Tooltips for disabled buttons should not be dimmed. Closes #70.

Aleksander Nowodzinski 8 years ago
parent
commit
83e36faeae

+ 7 - 0
packages/ckeditor5-theme-lark/theme/components/button.scss

@@ -66,6 +66,13 @@ a.ck-button {
 		float: left;
 		float: left;
 	}
 	}
 
 
+	// https://github.com/ckeditor/ckeditor5-theme-lark/issues/70
+	&.ck-disabled {
+		@include ck-button-icon {
+			@include ck-disabled;
+		}
+	}
+
 	&.ck-button_with-text {
 	&.ck-button_with-text {
 		padding: ck-spacing( 'small' ) ck-spacing();
 		padding: ck-spacing( 'small' ) ck-spacing();
 
 

+ 9 - 0
packages/ckeditor5-theme-lark/theme/components/dropdown.scss

@@ -14,6 +14,15 @@
 		// A space to accommodate the triangle.
 		// A space to accommodate the triangle.
 		padding-right: 2 * ck-spacing();
 		padding-right: 2 * ck-spacing();
 
 
+		// https://github.com/ckeditor/ckeditor5-theme-lark/issues/70
+		&.ck-disabled {
+			@include ck-button-colors( 'background', 10 );
+
+			.ck-button__label {
+				@include ck-disabled;
+			}
+		}
+
 		// #23
 		// #23
 		.ck-button__label {
 		.ck-button__label {
 			width: 7em;
 			width: 7em;

+ 7 - 2
packages/ckeditor5-theme-lark/theme/helpers/_states.scss

@@ -12,10 +12,15 @@ $ck-focus-outer-shadow: 0 0 3px 2px lighten( ck-color( 'focus' ), 10% );
 $ck-focus-ring: 1px solid ck-color( 'focus' );
 $ck-focus-ring: 1px solid ck-color( 'focus' );
 
 
 /**
 /**
+ * An opacity value of disabled UI item
+ */
+$ck-disabled-opacity: .5;
+
+/**
  * A class which indicates that an element holding it is disabled.
  * A class which indicates that an element holding it is disabled.
  */
  */
-.ck-disabled {
-	opacity: .5;
+@mixin ck-disabled {
+	opacity: $ck-disabled-opacity;
 }
 }
 
 
 /**
 /**