Merge branch 'release-v1.0.5'
This commit is contained in:
commit
4ca90b16c3
3 changed files with 10 additions and 28 deletions
|
@ -1,3 +1,10 @@
|
||||||
|
Changes in [1.0.5](https://github.com/matrix-org/matrix-react-sdk/releases/tag/v1.0.5) (2019-03-21)
|
||||||
|
===================================================================================================
|
||||||
|
[Full Changelog](https://github.com/matrix-org/matrix-react-sdk/compare/v1.0.4...v1.0.5)
|
||||||
|
|
||||||
|
* Hotfix: disable typing notifs jumping prevention for now
|
||||||
|
[\#2811](https://github.com/matrix-org/matrix-react-sdk/pull/2811)
|
||||||
|
|
||||||
Changes in [1.0.4](https://github.com/matrix-org/matrix-react-sdk/releases/tag/v1.0.4) (2019-03-18)
|
Changes in [1.0.4](https://github.com/matrix-org/matrix-react-sdk/releases/tag/v1.0.4) (2019-03-18)
|
||||||
===================================================================================================
|
===================================================================================================
|
||||||
[Full Changelog](https://github.com/matrix-org/matrix-react-sdk/compare/v1.0.4-rc.1...v1.0.4)
|
[Full Changelog](https://github.com/matrix-org/matrix-react-sdk/compare/v1.0.4-rc.1...v1.0.4)
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
{
|
{
|
||||||
"name": "matrix-react-sdk",
|
"name": "matrix-react-sdk",
|
||||||
"version": "1.0.4",
|
"version": "1.0.5",
|
||||||
"description": "SDK for matrix.org using React",
|
"description": "SDK for matrix.org using React",
|
||||||
"author": "matrix.org",
|
"author": "matrix.org",
|
||||||
"repository": {
|
"repository": {
|
||||||
|
|
|
@ -216,8 +216,6 @@ module.exports = React.createClass({
|
||||||
this._lastSetScroll = undefined;
|
this._lastSetScroll = undefined;
|
||||||
}
|
}
|
||||||
|
|
||||||
this._checkBlockShrinking();
|
|
||||||
|
|
||||||
this.props.onScroll(ev);
|
this.props.onScroll(ev);
|
||||||
|
|
||||||
this.checkFillState();
|
this.checkFillState();
|
||||||
|
@ -234,7 +232,6 @@ module.exports = React.createClass({
|
||||||
// where it ought to be, and set off pagination requests if necessary.
|
// where it ought to be, and set off pagination requests if necessary.
|
||||||
checkScroll: function() {
|
checkScroll: function() {
|
||||||
this._restoreSavedScrollState();
|
this._restoreSavedScrollState();
|
||||||
this._checkBlockShrinking();
|
|
||||||
this.checkFillState();
|
this.checkFillState();
|
||||||
},
|
},
|
||||||
|
|
||||||
|
@ -691,36 +688,14 @@ module.exports = React.createClass({
|
||||||
* jumping when the typing indicator gets replaced by a smaller message.
|
* jumping when the typing indicator gets replaced by a smaller message.
|
||||||
*/
|
*/
|
||||||
blockShrinking: function() {
|
blockShrinking: function() {
|
||||||
const messageList = this.refs.itemlist;
|
// Disabled for now because of https://github.com/vector-im/riot-web/issues/9205
|
||||||
if (messageList) {
|
|
||||||
const currentHeight = messageList.clientHeight;
|
|
||||||
messageList.style.minHeight = `${currentHeight}px`;
|
|
||||||
}
|
|
||||||
},
|
},
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Clear the previously set min height
|
* Clear the previously set min height
|
||||||
*/
|
*/
|
||||||
clearBlockShrinking: function() {
|
clearBlockShrinking: function() {
|
||||||
const messageList = this.refs.itemlist;
|
// Disabled for now because of https://github.com/vector-im/riot-web/issues/9205
|
||||||
if (messageList) {
|
|
||||||
messageList.style.minHeight = null;
|
|
||||||
}
|
|
||||||
},
|
|
||||||
|
|
||||||
_checkBlockShrinking: function() {
|
|
||||||
const sn = this._getScrollNode();
|
|
||||||
const scrollState = this.scrollState;
|
|
||||||
if (!scrollState.stuckAtBottom) {
|
|
||||||
const spaceBelowViewport = sn.scrollHeight - (sn.scrollTop + sn.clientHeight);
|
|
||||||
// only if we've scrolled up 200px from the bottom
|
|
||||||
// should we clear the min-height used by the typing notifications,
|
|
||||||
// otherwise we might still see it jump as the whitespace disappears
|
|
||||||
// when scrolling up from the bottom
|
|
||||||
if (spaceBelowViewport >= 200) {
|
|
||||||
this.clearBlockShrinking();
|
|
||||||
}
|
|
||||||
}
|
|
||||||
},
|
},
|
||||||
|
|
||||||
render: function() {
|
render: function() {
|
||||||
|
|
Loading…
Reference in a new issue