Merge pull request #12318 from ckeditor/ck/12277-color-button-unneces…

…sary-tooltip Fix (font): Removed an unnecessary tooltip for the "Remove color" button. Closes #12277.
ckeditor · Aug 23, 2022 · 6655e82 · 6655e82
2 parents 037ac54 + 76ebb50
commit 6655e82
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/packages/ckeditor5-font/src/ui/colortableview.js b/packages/ckeditor5-font/src/ui/colortableview.js
@@ -321,7 +321,6 @@ export default class ColorTableView extends View {
 		buttonView.set( {
 			withText: true,
 			icon: icons.eraser,
-			tooltip: true,
 			label: this.removeButtonLabel
 		} );
 

diff --git a/packages/ckeditor5-font/tests/ui/colortableview.js b/packages/ckeditor5-font/tests/ui/colortableview.js
@@ -123,7 +123,7 @@ describe( 'ColorTableView', () => {
 			expect( colorTableView.selectedColor ).to.equal( 'white' );
 		} );
 
-		it( 'should set tooltip for the remove color button', () => {
+		it( 'should set label for the remove color button', () => {
 			expect( colorTableView.removeButtonLabel ).to.equal( 'Remove color' );
 		} );
 
@@ -255,7 +255,6 @@ describe( 'ColorTableView', () => {
 		it( 'should have proper settings', () => {
 			expect( removeButton.withText ).to.be.true;
 			expect( removeButton.icon ).to.equal( removeButtonIcon );
-			expect( removeButton.tooltip ).to.be.true;
 			expect( removeButton.label ).to.equal( 'Remove color' );
 		} );