Преглед на файлове

Merge pull request #83 from ckeditor/t/82

Other: Removed tick symbol from active list item, used inverted background and text colors instead. Closes #82.
Szymon Kupś преди 8 години
родител
ревизия
b66ea88dc5
променени са 2 файла, в които са добавени 9 реда и са изтрити 22 реда
  1. 8 22
      packages/ckeditor5-theme-lark/theme/components/list.scss
  2. 1 0
      packages/ckeditor5-theme-lark/theme/helpers/_colors.scss

+ 8 - 22
packages/ckeditor5-theme-lark/theme/components/list.scss

@@ -2,28 +2,17 @@
 // For licensing, see LICENSE.md or http://ckeditor.com/license
 
 .ck-list {
+	@include ck-rounded-corners();
+	@include ck-unselectable();
+
 	list-style-type: none;
 	background: ck-color( 'background' );
 
-	@include ck-rounded-corners();
-
 	&__item {
-		padding: ck-spacing( 'medium' ) ck-spacing( 'medium' ) ck-spacing( 'medium' ) 0;
-		cursor: pointer;
+		padding: ck-spacing( 'medium' );
+		cursor: default;
 		min-width: 12em;
 
-		// NOTE: Using absolute sizes here because otherwise each
-		// item would indent in a different way.
-		&::before {
-			content: "";
-			display: inline-block;
-			width: $ck-font-size-base;
-			font-weight: normal;
-			font-size: $ck-font-size-base;
-			text-align: center;
-			margin: 0 $ck-font-size-base/2;
-		}
-
 		&:hover,
 		&:focus {
 			background: ck-color( 'foreground' );
@@ -41,15 +30,12 @@
 		}
 
 		&_active {
-			background: ck-color( 'foreground', -5 );
-
-			&::before {
-				content: "✓";
-			}
+			background: ck-color( 'active' );
+			color: ck-color( 'background' );
 
 			&:hover,
 			&:focus {
-				background: ck-color( 'foreground', -10 );
+				background: ck-color( 'active', -10 );
 			}
 		}
 	}

+ 1 - 0
packages/ckeditor5-theme-lark/theme/helpers/_colors.scss

@@ -23,6 +23,7 @@ $_ck-colors: (
 	'focus': #6ab5f9,
 	'foreground': #f7f7f7,
 	'text': #333,
+	'active': #1A8BF1,
 	'shadow': rgba(0,0,0,.2)
 );