Add option to change the size of images/videos in the timeline

res/css/_common.scss

@@ -32,6 +32,7 @@ $slider-selection-dot-size: 2.4em;
 
 $container-border-width: 8px;
 
+$timeline-image-boarder-radius: 8px;
 :root {
  font-size: 10px;
 

res/css/_components.scss

@@ -252,6 +252,7 @@
 @import "./views/settings/_E2eAdvancedPanel.scss";
 @import "./views/settings/_EmailAddresses.scss";
 @import "./views/settings/_FontScalingPanel.scss";
+@import "./views/settings/_ImageSizePanel.scss";
 @import "./views/settings/_IntegrationManager.scss";
 @import "./views/settings/_JoinRuleSettings.scss";
 @import "./views/settings/_LayoutSwitcher.scss";

res/css/views/messages/_MImageBody.scss

@@ -1,5 +1,6 @@
 /*
 Copyright 2015, 2016 OpenMarket Ltd
+Copyright 2021 The Matrix.org Foundation C.I.C.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -14,7 +15,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-$timelineImageBorderRadius: 4px;
+$timeline-image-boarder-radius: 8px;
 
 .mx_MImageBody_thumbnail--blurhash {
  position: absolute;
@@ -24,17 +25,18 @@ $timelineImageBorderRadius: 4px;
 
 .mx_MImageBody_thumbnail {
  object-fit: contain;
- border-radius: $timelineImageBorderRadius;
+ border-radius: $timeline-image-boarder-radius;
 
  display: flex;
  justify-content: center;
  align-items: center;
  height: 100%;
  width: 100%;
 
+ // this is needed so that the Blurhash can get have rounded corners without beeing the correct size during loading.
+ overflow: hidden;
  .mx_Blurhash > canvas {
  animation: mx--anim-pulse 1.75s infinite cubic-bezier(.4, 0, .6, 1);
- border-radius: $timelineImageBorderRadius;
  }
 
  .mx_no-image-placeholder {

res/css/views/messages/_MVideoBody.scss

@@ -1,5 +1,5 @@
 /*
-Copyright 2020 The Matrix.org Foundation C.I.C.
+Copyright 2020 - 2021 The Matrix.org Foundation C.I.C.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -18,6 +18,6 @@ span.mx_MVideoBody {
  video.mx_MVideoBody {
  max-width: 100%;
  height: auto;
- border-radius: 4px;
+ border-radius: $timeline-image-boarder-radius;
  }
 }

res/css/views/settings/_ImageSizePanel.scss

@@ -0,0 +1,47 @@
+/*
+Copyright 2021 The Matrix.org Foundation C.I.C.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+.mx_ImageSizePanel {
+ color: $primary-content;
+
+ .mx_ImageSizePanel_radios {
+ display: flex;
+ margin-top: 16px; // move away from header a bit
+
+ > label {
+ margin-right: 68px; // keep the boxes separate
+ cursor: pointer;
+ }
+
+ .mx_ImageSizePanel_size {
+ background-color: $quinary-content;
+ mask-repeat: no-repeat;
+ mask-size: 221px;
+ mask-position: center;
+ width: 221px;
+ height: 148px;
+ margin-bottom: 14px; // move radio button away from bottom edge a bit
+
+ &.mx_ImageSizePanel_sizeDefault {
+ mask: url("$(res)/img/element-icons/settings/img-size-normal.svg");
+ }
+
+ &.mx_ImageSizePanel_sizeLarge {
+ mask: url("$(res)/img/element-icons/settings/img-size-large.svg");
+ }
+ }
+ }
+}

src/components/structures/FilePanel.tsx

@@ -36,7 +36,7 @@ import ResizeNotifier from '../../utils/ResizeNotifier';
 import TimelinePanel from "./TimelinePanel";
 import Spinner from "../views/elements/Spinner";
 import { TileShape } from '../views/rooms/EventTile';
-import { Layout } from "../../settings/Layout";
+import { Layout } from "../../settings/enums/Layout";
 import RoomContext, { TimelineRenderingType } from '../../contexts/RoomContext';
 
 import { logger } from "matrix-js-sdk/src/logger";

src/components/structures/MessagePanel.tsx

@@ -27,7 +27,7 @@ import { wantsDateSeparator } from '../../DateUtils';
 import { MatrixClientPeg } from '../../MatrixClientPeg';
 import SettingsStore from '../../settings/SettingsStore';
 import RoomContext, { TimelineRenderingType } from "../../contexts/RoomContext";
-import { Layout } from "../../settings/Layout";
+import { Layout } from "../../settings/enums/Layout";
 import { _t } from "../../languageHandler";
 import EventTile, { haveTileForEvent, IReadReceiptProps, TileShape } from "../views/rooms/EventTile";
 import { hasText } from "../../TextForEvent";

src/components/structures/NotificationPanel.tsx

@@ -23,7 +23,7 @@ import { replaceableComponent } from "../../utils/replaceableComponent";
 import TimelinePanel from "./TimelinePanel";
 import Spinner from "../views/elements/Spinner";
 import { TileShape } from "../views/rooms/EventTile";
-import { Layout } from "../../settings/Layout";
+import { Layout } from "../../settings/enums/Layout";
 import RoomContext, { TimelineRenderingType } from "../../contexts/RoomContext";
 
 import { logger } from "matrix-js-sdk/src/logger";

src/components/structures/RoomView.tsx

@@ -44,7 +44,7 @@ import RoomViewStore from '../../stores/RoomViewStore';
 import RoomScrollStateStore, { ScrollState } from '../../stores/RoomScrollStateStore';
 import WidgetEchoStore from '../../stores/WidgetEchoStore';
 import SettingsStore from "../../settings/SettingsStore";
-import { Layout } from "../../settings/Layout";
+import { Layout } from "../../settings/enums/Layout";
 import AccessibleButton from "../views/elements/AccessibleButton";
 import RightPanelStore from "../../stores/RightPanelStore";
 import { haveTileForEvent } from "../views/rooms/EventTile";

src/components/structures/ThreadPanel.tsx

@@ -27,7 +27,7 @@ import { ContextMenuButton } from '../../accessibility/context_menu/ContextMenuB
 import ContextMenu, { ChevronFace, useContextMenu } from './ContextMenu';
 import RoomContext, { TimelineRenderingType } from '../../contexts/RoomContext';
 import TimelinePanel from './TimelinePanel';
-import { Layout } from '../../settings/Layout';
+import { Layout } from '../../settings/enums/Layout';
 import { useEventEmitter } from '../../hooks/useEventEmitter';
 import AccessibleButton from '../views/elements/AccessibleButton';
 import { TileShape } from '../views/rooms/EventTile';

src/components/structures/ThreadView.tsx

@@ -27,7 +27,7 @@ import ResizeNotifier from '../../utils/ResizeNotifier';
 import { TileShape } from '../views/rooms/EventTile';
 import MessageComposer from '../views/rooms/MessageComposer';
 import { RoomPermalinkCreator } from '../../utils/permalinks/Permalinks';
-import { Layout } from '../../settings/Layout';
+import { Layout } from '../../settings/enums/Layout';
 import TimelinePanel from './TimelinePanel';
 import dis from "../../dispatcher/dispatcher";
 import { ActionPayload } from '../../dispatcher/payloads';

src/components/structures/TimelinePanel.tsx

@@ -25,7 +25,7 @@ import { EventType, RelationType } from 'matrix-js-sdk/src/@types/event';
 import { SyncState } from 'matrix-js-sdk/src/sync.api';
 
 import SettingsStore from "../../settings/SettingsStore";
-import { Layout } from "../../settings/Layout";
+import { Layout } from "../../settings/enums/Layout";
 import { _t } from '../../languageHandler';
 import { MatrixClientPeg } from "../../MatrixClientPeg";
 import RoomContext from "../../contexts/RoomContext";

src/components/views/dialogs/ForwardDialog.tsx

@@ -25,7 +25,7 @@ import { _t } from "../../../languageHandler";
 import dis from "../../../dispatcher/dispatcher";
 import { useSettingValue, useFeatureEnabled } from "../../../hooks/useSettings";
 import { UIFeature } from "../../../settings/UIFeature";
-import { Layout } from "../../../settings/Layout";
+import { Layout } from "../../../settings/enums/Layout";
 import { IDialogProps } from "./IDialogProps";
 import BaseDialog from "./BaseDialog";
 import { avatarUrlForUser } from "../../../Avatar";

src/components/views/elements/EventListSummary.tsx

@@ -22,7 +22,7 @@ import MemberAvatar from '../avatars/MemberAvatar';
 import { _t } from '../../../languageHandler';
 import { useStateToggle } from "../../../hooks/useStateToggle";
 import AccessibleButton from "./AccessibleButton";
-import { Layout } from '../../../settings/Layout';
+import { Layout } from '../../../settings/enums/Layout';
 
 interface IProps {
  // An array of member events to summarise

src/components/views/elements/EventTilePreview.tsx

@@ -22,7 +22,7 @@ import { RoomMember } from 'matrix-js-sdk/src/models/room-member';
 import * as Avatar from '../../../Avatar';
 import EventTile from '../rooms/EventTile';
 import SettingsStore from "../../../settings/SettingsStore";
-import { Layout } from "../../../settings/Layout";
+import { Layout } from "../../../settings/enums/Layout";
 import { UIFeature } from "../../../settings/UIFeature";
 import { replaceableComponent } from "../../../utils/replaceableComponent";
 import Spinner from './Spinner';

src/components/views/elements/MemberEventListSummary.tsx

@@ -31,7 +31,7 @@ import { Action } from '../../../dispatcher/actions';
 import { SetRightPanelPhasePayload } from '../../../dispatcher/payloads/SetRightPanelPhasePayload';
 import { jsxJoin } from '../../../utils/ReactUtils';
 import { EventType } from 'matrix-js-sdk/src/@types/event';
-import { Layout } from '../../../settings/Layout';
+import { Layout } from '../../../settings/enums/Layout';
 
 const onPinnedMessagesClick = (): void => {
  defaultDispatcher.dispatch<SetRightPanelPhasePayload>({

src/components/views/elements/ReplyChain.tsx

@@ -23,7 +23,7 @@ import dis from '../../../dispatcher/dispatcher';
 import { MatrixEvent } from 'matrix-js-sdk/src/models/event';
 import { makeUserPermalink, RoomPermalinkCreator } from "../../../utils/permalinks/Permalinks";
 import SettingsStore from "../../../settings/SettingsStore";
-import { Layout } from "../../../settings/Layout";
+import { Layout } from "../../../settings/enums/Layout";
 import escapeHtml from "escape-html";
 import MatrixClientContext from "../../../contexts/MatrixClientContext";
 import { getUserNameColorClass } from "../../../utils/FormattingUtils";

src/components/views/messages/MImageBody.tsx

@@ -35,6 +35,7 @@ import classNames from 'classnames';
 import { CSSTransition, SwitchTransition } from 'react-transition-group';
 
 import { logger } from "matrix-js-sdk/src/logger";
+import { ImageSize, suggestedSize as suggestedImageSize } from "../../../settings/enums/ImageSize";
 
 interface IState {
  decryptedUrl?: string;
@@ -58,6 +59,7 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
  private unmounted = true;
  private image = createRef<HTMLImageElement>();
  private timeout?: number;
+ private sizeWatcher: string;
 
  constructor(props: IBodyProps) {
  super(props);
@@ -317,12 +319,17 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
  }
  }, 150);
  }
+
+ this.sizeWatcher = SettingsStore.watchSetting("Images.size", null, () => {
+ this.forceUpdate(); // we don't really have a reliable thing to update, so just update the whole thing
+ });
  }
 
  componentWillUnmount() {
  this.unmounted = true;
  this.context.removeListener('sync', this.onClientSync);
  this.clearBlurhashTimeout();
+ SettingsStore.unwatchSetting(this.sizeWatcher);
  }
 
  protected messageContent(
@@ -367,11 +374,25 @@ export default class MImageBody extends React.Component<IBodyProps, IState> {
  infoHeight = this.state.loadedImageDimensions.naturalHeight;
  }
 
- // The maximum height of the thumbnail as it is rendered as an <img>
- const maxHeight = forcedHeight || Math.min((this.props.maxImageHeight || 600), infoHeight);
- // The maximum width of the thumbnail, as dictated by its natural
- // maximum height.
- const maxWidth = infoWidth * maxHeight / infoHeight;
+ // The maximum size of the thumbnail as it is rendered as an <img>
+ // check for any height constraints
+ const imageSize = SettingsStore.getValue("Images.size") as ImageSize;
+ const suggestedAndPossibleWidth = Math.min(suggestedImageSize(imageSize).w, infoWidth);
+ const aspectRatio = infoWidth / infoHeight;
+
+ let maxWidth;
+ let maxHeight;
+ const maxHeightConstraint = forcedHeight || this.props.maxImageHeight || undefined;
+ if (maxHeightConstraint && maxHeightConstraint * aspectRatio < suggestedAndPossibleWidth) {
+ // width is dictated by the maximum height that was defined by the props or the function param `forcedHeight`
+ maxWidth = maxHeightConstraint * aspectRatio;
+ // there is no need to check for infoHeight here since this is done with `maxHeightConstraint * aspectRatio < suggestedAndPossibleWidth`
+ maxHeight = maxHeightConstraint;
+ } else {
+ // height is dictated by suggestedWidth (based on the Image.size setting)
+ maxWidth = suggestedAndPossibleWidth;
+ maxHeight = suggestedAndPossibleWidth / aspectRatio;
+ }
 
  let img = null;
  let placeholder = null;