IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-12-01 13:09:33 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: ensures nothing is triggering rendering loop in after render (#6784)

Browse Source
This commit is contained in:
Joffrey JAFFEUX 2018-12-17 12:15:03 +01:00 committed by GitHub
parent 01cdbd3a13
commit 6ee3900791
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 15 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/assets/javascripts/select-kit/components/select-kit.js.es6
View File

@ -25,10 +25,6 @@ export default Ember.Component.extend(
"isExpanded",
"isDisabled",
"isHidden",
"isAbove",
"isBelow",
"isLeftAligned",
"isRightAligned",
"hasSelection",
"hasReachedMaximum",
"hasReachedMinimum"

20
app/assets/javascripts/select-kit/mixins/dom-helpers.js.es6
View File

@ -69,8 +69,10 @@ export default Ember.Mixin.create({
},
_adjustPosition() {
this._applyFixedPosition();
if (this.get("isExpanded")) {
this._applyDirection();
}
this._applyFixedPosition();
this._positionWrapper();
},
@ -197,11 +199,15 @@ export default Ember.Mixin.create({
parentWidth - marginToEdge - bodyWidth + this.get("horizontalOffset") >
0;
if (enoughMarginToOppositeEdge) {
this.setProperties({ isLeftAligned: true, isRightAligned: false });
this.$()
.addClass("is-left-aligned")
.removeClass("is-right-aligned");
options.left = this.get("horizontalOffset");
options.right = "unset";
} else {
this.setProperties({ isLeftAligned: false, isRightAligned: true });
this.$()
.addClass("is-right-aligned")
.removeClass("is-left-aligned");
options.left = "unset";
options.right = this.get("horizontalOffset");
}
@ -214,10 +220,14 @@ export default Ember.Mixin.create({
const headerHeight = this._computedStyle(this.$header()[0], "height");
if (hasBelowSpace || (!hasBelowSpace && !hasAboveSpace)) {
this.setProperties({ isBelow: true, isAbove: false });
this.$()
.addClass("is-below")
.removeClass("is-above");
options.top = headerHeight + this.get("verticalOffset");
} else {
this.setProperties({ isBelow: false, isAbove: true });
this.$()
.addClass("is-above")
.removeClass("is-below");
options.bottom = headerHeight + this.get("verticalOffset");
}