-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Bug 1211395 - Performance regression in Message #32458
Bug 1211395 - Performance regression in Message #32458
Conversation
if (!this.selectionHandler) { | ||
LazyLoader.load('/views/shared/js/selection_handler.js', () => { | ||
LazyLoader.load([ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this part could be wrapped into utils. BTW it's amazing that we have 3 different edit-mode.css for an edit form...
79c96b0
to
6a1ba3e
Compare
_lazyLoadInit: function() { | ||
var lazyLoadPromise = LazyLoader.load(this._lazyLoadScripts).then(() => { | ||
var lazyLoadFiles = this._lazyLoadScripts.concat(this._lazyLoadStyles); | ||
var lazyLoadPromise = LazyLoader.load(lazyLoadFiles).then(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
optional nit: you probably can do load([...this._lazyLoadScripts, ...this._lazyLoadStyles])
6a1ba3e
to
f7d5bdf
Compare
f7d5bdf
to
ec87030
Compare
@@ -611,6 +613,18 @@ suite('navigation >', function() { | |||
}).then(done, done); | |||
}); | |||
}); | |||
|
|||
suite('setReady', function() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This trick is only for non-split view, so only add it in the non-split test suite.
this needs a simple rebase |
ec87030
to
409c04a
Compare
Rebased! |
@@ -6681,7 +6683,10 @@ suite('conversation.js >', function() { | |||
|
|||
test('enables the font-fit algorithm in headers', function() { | |||
assert.isFalse(header.hasAttribute('no-font-fit')); | |||
assert.isFalse(editHeader.hasAttribute('no-font-fit')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mmm this test should move in the edit mode tests, I think ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok I see it's in sms_test...
428a8f6
to
f1c8369
Compare
Bug 1211395 - Performance regression in Message, r=julienw azasypkin
No description provided.