Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Use MessageActionBar style declarations on pinned message card #8757

Merged
merged 7 commits into from
Jun 8, 2022
Merged

Use MessageActionBar style declarations on pinned message card #8757

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f475ac1
Refer mx_MessageActionBar on pinned messages card
luixxiul Jun 5, 2022
44324a4
Use variables for normalization
luixxiul Jun 5, 2022
e1851f7
Set class name
luixxiul Jun 5, 2022
5e45f12
Merge branch 'develop' into PinnedMessagesCard_MessageActionBar
luixxiul Jun 7, 2022
b9eaedc
Readd the comment
luixxiul Jun 8, 2022
c53cea8
Use Heading size='h4'
luixxiul Jun 8, 2022
3f5f1d3
Merge branch 'develop' into PinnedMessagesCard_MessageActionBar
luixxiul Jun 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions res/css/views/messages/_MessageActionBar.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ limitations under the License.
.mx_MessageActionBar {
--MessageActionBar-size-button: 28px;
--MessageActionBar-size-box: 32px; // 28px + 2px (margin) * 2
--MessageActionBar-item-hover-background: $panel-actions;
--MessageActionBar-item-hover-borderRadius: 6px;
--MessageActionBar-item-hover-zIndex: 1;

position: absolute;
visibility: hidden;
Expand Down Expand Up @@ -59,9 +62,9 @@ limitations under the License.
margin: 2px;

&:hover {
background: $panel-actions;
border-radius: 6px;
z-index: 1;
background: var(--MessageActionBar-item-hover-background);
border-radius: var(--MessageActionBar-item-hover-borderRadius);
z-index: var(--MessageActionBar-item-hover-zIndex);
}
}
}
Expand Down
36 changes: 14 additions & 22 deletions res/css/views/right_panel/_PinnedMessagesCard.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,51 +33,43 @@ limitations under the License.
}
}

.mx_PinnedMessagesCard_empty {
.mx_PinnedMessagesCard_empty_wrapper {
display: flex;
height: 100%;

> div {
.mx_PinnedMessagesCard_empty {
height: max-content;
text-align: center;
margin: auto 40px;

.mx_PinnedMessagesCard_MessageActionBar {
pointer-events: none;
display: flex;
height: 32px;
line-height: $font-24px;
border-radius: 8px;
background: $background;
border: 1px solid $input-border-color;
padding: 1px;
width: max-content;
margin: 0 auto;
box-sizing: border-box;

.mx_MessageActionBar_maskButton {
display: inline-block;
position: relative;
// Cancel the default values for non-interactivity
position: unset;
visibility: visible;
cursor: unset;

&::before {
content: unset;
}

.mx_MessageActionBar_optionsButton {
background: $panel-actions;
border-radius: 6px;
z-index: 1;
background: var(--MessageActionBar-item-hover-background);
border-radius: var(--MessageActionBar-item-hover-borderRadius);
z-index: var(--MessageActionBar-item-hover-zIndex);

&::after {
background-color: $primary-content;
}
}
}

> h2 {
font-weight: $font-semi-bold;
font-size: $font-15px;
line-height: $font-24px;
.mx_PinnedMessagesCard_empty_header {
color: $primary-content;
margin-top: 24px;
margin-bottom: 20px;
margin-block: $spacing-24 $spacing-20;
}

> span {
Expand Down
9 changes: 5 additions & 4 deletions src/components/views/right_panel/PinnedMessagesCard.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ import PinnedEventTile from "../rooms/PinnedEventTile";
import { useRoomState } from "../../../hooks/useRoomState";
import RoomContext, { TimelineRenderingType } from "../../../contexts/RoomContext";
import { ReadPinsEventId } from "./types";
import Heading from '../typography/Heading';

interface IProps {
room: Room;
Expand Down Expand Up @@ -154,16 +155,16 @@ const PinnedMessagesCard = ({ room, onClose }: IProps) => {
/>
));
} else {
content = <div className="mx_PinnedMessagesCard_empty">
<div>
content = <div className="mx_PinnedMessagesCard_empty_wrapper">
<div className="mx_PinnedMessagesCard_empty">
{ /* XXX: We reuse the classes for simplicity, but deliberately not the components for non-interactivity. */ }
luixxiul marked this conversation as resolved.
Show resolved Hide resolved
<div className="mx_PinnedMessagesCard_MessageActionBar">
<div className="mx_MessageActionBar mx_PinnedMessagesCard_MessageActionBar">
<div className="mx_MessageActionBar_maskButton mx_MessageActionBar_reactButton" />
<div className="mx_MessageActionBar_maskButton mx_MessageActionBar_replyButton" />
<div className="mx_MessageActionBar_maskButton mx_MessageActionBar_optionsButton" />
</div>

<h2>{ _t("Nothing pinned, yet") }</h2>
<Heading size="h4" className="mx_PinnedMessagesCard_empty_header">{ _t("Nothing pinned, yet") }</Heading>
{ _t("If you have permissions, open the menu on any message and select " +
"<b>Pin</b> to stick them here.", {}, {
b: sub => <b>{ sub }</b>,
Expand Down