Skip to content

Commit

Permalink
Merge pull request xtermjs#1329 from Tyriar/1319_bang_operator
Browse files Browse the repository at this point in the history
Use bang operator for array methods
  • Loading branch information
Tyriar authored Mar 21, 2018
2 parents b85083c + a59af66 commit 4ab79c9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/AccessibilityManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -121,10 +121,10 @@ export class AccessibilityManager implements IDisposable {
let bottomBoundaryElement: HTMLElement;
if (position === BoundaryPosition.Top) {
topBoundaryElement = boundaryElement;
bottomBoundaryElement = <HTMLElement>this._rowElements.pop();
bottomBoundaryElement = this._rowElements.pop()!;
this._rowContainer.removeChild(bottomBoundaryElement);
} else {
topBoundaryElement = <HTMLElement>this._rowElements.shift();
topBoundaryElement = this._rowElements.shift()!;
bottomBoundaryElement = boundaryElement;
this._rowContainer.removeChild(topBoundaryElement);
}
Expand Down Expand Up @@ -170,7 +170,7 @@ export class AccessibilityManager implements IDisposable {
}
// Shrink rows as required
while (this._rowElements.length > rows) {
this._rowContainer.removeChild(<HTMLElement>this._rowElements.pop());
this._rowContainer.removeChild(this._rowElements.pop()!);
}

// Add bottom boundary listener
Expand Down

0 comments on commit 4ab79c9

Please sign in to comment.