mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix position of lightbox arrows for a video

Browse source
This commit is contained in:
Fedor Indutny 2023-05-22 22:24:33 +02:00 committed by GitHub
parent 759c61d06e
commit 99011b0a56
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 0 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
stylesheets/components/Lightbox.scss
View file

@ -256,17 +256,6 @@
}
}
&__container--has-controls {
.Lightbox__nav-next,
.Lightbox__nav-prev {
// Give some room for <video/>'s controls.
$controls-height: 64px;
padding-top: $controls-height;
height: calc(100% - $controls-height);
}
}
&__controls {
display: flex;
gap: 32px;

4
ts/components/Lightbox.tsx
View file

@ -484,7 +484,6 @@ export function Lightbox({
const caption = attachment?.caption;
let content: JSX.Element;
let hasControls = false;
if (!contentType) {
content = <>{children}</>;
} else {
@ -551,8 +550,6 @@ export function Lightbox({
<source src={objectURL} />
</video>
);
hasControls = true;
} else if (isUnsupportedImageType || isUnsupportedVideoType) {
content = (
<button
@ -590,7 +587,6 @@ export function Lightbox({
<div
className={classNames('Lightbox Lightbox__container', {
'Lightbox__container--zoom': isZoomed,
'Lightbox__container--has-controls': hasControls,
})}
onClick={(event: React.MouseEvent<HTMLDivElement>) => {
event.stopPropagation();