PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Fix border around persisted widgets

Browse source 
The PersistedElement had zero height so the border ended up in the
wrong place. Add a wrapper to fix the height back.

Fixes https://github.com/vector-im/riot-web/issues/7030
Fixes https://github.com/vector-im/riot-web/issues/7029
This commit is contained in:
David Baker 2018-07-18 11:52:57 +01:00
parent 0a432dfb8f
commit f868cd6699
2 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
res/css/views/rooms/_AppsDrawer.scss
View file

@ -61,6 +61,10 @@ limitations under the License.
background-color: $dialog-background-bg-color; background-color: $dialog-background-bg-color;
} }
.mx_AppTile_persistedWrapper {
height: 280px;
}
.mx_AppTile:last-child { .mx_AppTile:last-child {
margin-right: 1px; margin-right: 1px;
} }

10
src/components/views/elements/AppTile.js
View file

@ -569,9 +569,13 @@ export default class AppTile extends React.Component {
// re-mounted later when we do. // re-mounted later when we do.
if (this.props.whitelistCapabilities.includes('m.always_on_screen')) { if (this.props.whitelistCapabilities.includes('m.always_on_screen')) {
const PersistedElement = sdk.getComponent("elements.PersistedElement"); const PersistedElement = sdk.getComponent("elements.PersistedElement");
appTileBody = <PersistedElement persistKey={this._persistKey}> // Also wrap the PersistedElement in a div to fix the height, otherwise
{appTileBody} // AppTile's border is in the wrong place
</PersistedElement>; appTileBody = <div className="mx_AppTile_persistedWrapper">
<PersistedElement persistKey={this._persistKey}>
{appTileBody}
</PersistedElement>
</div>;
} }
} }
} else { } else {