beenull/browsh
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Text builder optimisations from @tobimensch

Browse source
This commit is contained in:
Thomas Buckley-Houston 2018-06-15 11:15:01 +08:00
parent 92de166b77
commit 1b42630b7f
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
webext/src/dom/text_builder.js
View file

@ -99,7 +99,7 @@ export default class extends utils.mixins(CommonMixin) {
}
// Ignore nodes with only whitespace
if (/^\s+$/.test(node.textContent) || node.textContent === '') {
if (node.textContent.trim().length === 0) {
return false;
}
@ -190,10 +190,10 @@ export default class extends utils.mixins(CommonMixin) {
if (this._text.charAt(0) === " ") {
this._text = this._text.substring(1, this._text.length);
}
}
// Remove whitespace at the end
if (this._text.charAt(this._text.length - 1) === " ") {
this._text = this._text.substring(0, this._text.length - 1);
// Remove whitespace at the end
if (this._text.charAt(this._text.length - 1) === " ") {
this._text = this._text.substring(0, this._text.length - 1);
}
}
}
@ -321,7 +321,7 @@ export default class extends utils.mixins(CommonMixin) {
// of a DOM rectangle's box (`this._dom_box`).
_ignoreUnrenderedWhitespace() {
if (this._isNewLine()) {
if (/[\t\n\r ]+/.test(this._current_character)) this._stepToNextCharacter(false);
if (this._current_character.trim().length == 0) this._stepToNextCharacter(false);
}
}