Browse Source

Merge pull request #222 from ckeditor/cf/3055

Internal: Set `MergeCellCommand#value` before `#isEnabled` so it is available in `#isEnabled` observable callbacks.
Piotr Jasiun 6 years ago
parent
commit
52fab2c91e
1 changed files with 1 additions and 3 deletions
  1. 1 3
      packages/ckeditor5-table/src/commands/mergecellcommand.js

+ 1 - 3
packages/ckeditor5-table/src/commands/mergecellcommand.js

@@ -64,10 +64,8 @@ export default class MergeCellCommand extends Command {
 	refresh() {
 		const cellToMerge = this._getMergeableCell();
 
-		this.isEnabled = !!cellToMerge;
-		// In order to check if currently selected cell can be merged with one defined by #direction some computation are done beforehand.
-		// As such we can cache it as a command's value.
 		this.value = cellToMerge;
+		this.isEnabled = !!cellToMerge;
 	}
 
 	/**