Merge pull request #221 from ckeditor/t/220

Feature: Implemented placeholder in `InputTextView`. Closes #220.
ckeditor · May 8, 2017 · 5d91d18 · 5d91d18
2 parents 8ad3474 + 450cedb
commit 5d91d18
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/inputtext/inputtextview.js b/src/inputtext/inputtextview.js
@@ -38,6 +38,14 @@ export default class InputTextView extends View {
 		 */
 		this.set( 'id' );
 
+		/**
+		 * The `placeholder` attribute of the input.
+		 *
+		 * @observable
+		 * @member {String} #placeholder
+		 */
+		this.set( 'placeholder' );
+
 		const bind = this.bindTemplate;
 
 		this.template = new Template( {
@@ -48,7 +56,8 @@ export default class InputTextView extends View {
 					'ck-input',
 					'ck-input-text'
 				],
-				id: bind.to( 'id' )
+				id: bind.to( 'id' ),
+				placeholder: bind.to( 'placeholder' )
 			}
 		} );
 

diff --git a/tests/inputtext/inputtextview.js b/tests/inputtext/inputtextview.js
@@ -56,6 +56,16 @@ describe( 'InputTextView', () => {
 				expect( view.element.id ).to.equal( 'baz' );
 			} );
 		} );
+
+		describe( 'placeholder', () => {
+			it( 'should react on view#placeholder', () => {
+				expect( view.element.placeholder ).to.equal( '' );
+
+				view.placeholder = 'baz';
+
+				expect( view.element.placeholder ).to.equal( 'baz' );
+			} );
+		} );
 	} );
 
 	describe( 'select()', () => {