toki/mattermost-mobile
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Prevent onLayout if it was already run (#1316)

Browse source
This commit is contained in:
enahum 2017-12-22 16:10:20 -03:00 committed by GitHub
parent 2181168c9f
commit eb214ab1d8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 26 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

42
app/components/badge.js
View file

@ -33,6 +33,7 @@ export default class Badge extends PureComponent {
super(props);
this.mounted = false;
this.layoutReady = false;
}
componentWillMount() {
@ -49,6 +50,12 @@ export default class Badge extends PureComponent {
this.mounted = true;
}
componentWillReceiveProps(nextProps) {
if (nextProps.count !== this.props.count) {
this.layoutReady = false;
}
}
componentWillUnmount() {
this.mounted = false;
}
@ -66,24 +73,27 @@ export default class Badge extends PureComponent {
};
onLayout = (e) => {
const height = Math.max(e.nativeEvent.layout.height, this.props.minHeight);
const borderRadius = height / 2;
let width;
if (!this.layoutReady) {
const height = Math.max(e.nativeEvent.layout.height, this.props.minHeight);
const borderRadius = height / 2;
let width;
if (e.nativeEvent.layout.width <= e.nativeEvent.layout.height) {
width = e.nativeEvent.layout.height;
} else {
width = e.nativeEvent.layout.width + this.props.extraPaddingHorizontal;
}
width = Math.max(width, this.props.minWidth);
this.setNativeProps({
style: {
width,
height,
borderRadius,
opacity: 1
if (e.nativeEvent.layout.width <= e.nativeEvent.layout.height) {
width = e.nativeEvent.layout.height;
} else {
width = e.nativeEvent.layout.width + this.props.extraPaddingHorizontal;
}
});
width = Math.max(width, this.props.minWidth);
this.setNativeProps({
style: {
width,
height,
borderRadius,
opacity: 1
}
});
this.layoutReady = true;
}
};
renderText = () => {