Add CSS variable to make the UI gaps consistent and fix the resize handle position (#7234)
Co-authored-by: J. Ryan Stinnett <jryans@gmail.com>
This commit is contained in:
@@ -20,15 +20,6 @@ limitations under the License.
|
||||
z-index: 100;
|
||||
}
|
||||
|
||||
// We'd like to remove this, but this makes matrixchat's resizehandle's
|
||||
// negative margin & greater than its positive padding. If it's the same
|
||||
// or less, Safari gets confused about overflows somehow and
|
||||
// https://github.com/vector-im/element-web/issues/19863 happens.
|
||||
.mx_MatrixChat > .mx_ResizeHandle.mx_ResizeHandle_horizontal {
|
||||
margin: 0 -10px 0 0;
|
||||
padding: 0 8px 0 0;
|
||||
}
|
||||
|
||||
.mx_ResizeHandle.mx_ResizeHandle_horizontal {
|
||||
margin: 0 -5px;
|
||||
padding: 0 5px;
|
||||
|
||||
@@ -18,19 +18,22 @@ limitations under the License.
|
||||
$MiniAppTileHeight: 200px;
|
||||
|
||||
.mx_AppsDrawer {
|
||||
margin: 5px;
|
||||
margin: $container-gap-width;
|
||||
margin-right: calc($container-gap-width / 2); // The left side gap is fully handled by this margin. To prohibit bleeding on webkit browser.
|
||||
margin-bottom: 0; // No bottom margin for the correct gap to the CallView below.
|
||||
margin-left: 13px; // 5+8px to compensate for the scroll bar padding on the right.
|
||||
position: relative;
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
overflow: hidden;
|
||||
flex-grow: 1;
|
||||
|
||||
&.mx_AppsDrawer_maximise {
|
||||
margin-bottom: $container-gap-width;
|
||||
}
|
||||
|
||||
.mx_AppsContainer_resizerHandleContainer {
|
||||
width: 100%;
|
||||
height: 10px;
|
||||
margin-top: -3px; // move it up so the interactions are slightly more comfortable
|
||||
display: block;
|
||||
position: relative;
|
||||
}
|
||||
@@ -44,7 +47,7 @@ $MiniAppTileHeight: 200px;
|
||||
|
||||
// This is positioned directly below frame
|
||||
position: absolute;
|
||||
bottom: 0 !important; // override from library
|
||||
bottom: 50% !important; // override from library
|
||||
|
||||
// We then render the pill handle in an ::after to keep it in the handle's
|
||||
// area without being a massive line across the screen
|
||||
@@ -53,8 +56,7 @@ $MiniAppTileHeight: 200px;
|
||||
position: absolute;
|
||||
border-radius: 3px;
|
||||
|
||||
// The combination of these two should make the pill 4px high
|
||||
top: 6px;
|
||||
height: 4px;
|
||||
bottom: 0;
|
||||
|
||||
// Together, these make the bar 64px wide
|
||||
@@ -89,7 +91,7 @@ $MiniAppTileHeight: 200px;
|
||||
}
|
||||
|
||||
.mx_AppsContainer_resizer {
|
||||
margin-bottom: 8px;
|
||||
margin-bottom: $container-gap-width;
|
||||
}
|
||||
|
||||
.mx_AppsContainer {
|
||||
|
||||
@@ -26,7 +26,9 @@ limitations under the License.
|
||||
|
||||
.mx_CallView_large {
|
||||
padding-bottom: 10px;
|
||||
margin: 5px 5px 5px 13px;
|
||||
margin: $container-gap-width;
|
||||
margin-right: calc($container-gap-width / 2); // The left side gap is fully handled by this margin. To prohibit bleeding on webkit browser.
|
||||
margin-bottom: 10px;
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
flex: 1;
|
||||
|
||||
@@ -19,7 +19,6 @@ limitations under the License.
|
||||
|
||||
.mx_CallViewForRoom_ResizeWrapper {
|
||||
display: flex;
|
||||
margin-bottom: 8px;
|
||||
|
||||
&:hover .mx_CallViewForRoom_ResizeHandle {
|
||||
// Need to use important to override element style attributes
|
||||
@@ -31,8 +30,6 @@ limitations under the License.
|
||||
|
||||
&::after {
|
||||
content: '';
|
||||
margin-top: 3px;
|
||||
|
||||
border-radius: 4px;
|
||||
|
||||
height: 4px;
|
||||
|
||||
Reference in New Issue
Block a user