Update feedback prompt to match designs

This commit is contained in:
Michael Telatynski 2021-07-28 17:56:57 +01:00
parent 6123ccb2eb
commit a5606c72de
4 changed files with 42 additions and 102 deletions

View file

@ -340,18 +340,12 @@ $SpaceRoomViewInnerWidth: 428px;
flex: 0; flex: 0;
} }
.mx_SpaceFeedbackPrompt_topRight { .mx_SpaceFeedbackPrompt {
padding: 7px; // 8px - 1px border padding: 7px; // 8px - 1px border
border: 1px solid $menu-border-color; border: 1px solid $menu-border-color;
border-radius: 8px; border-radius: 8px;
color: $secondary-fg-color; width: max-content;
font-size: $font-15px; margin: 0 0 -40px auto; // collapse its own height to not push other components down
line-height: $font-24px;
float: right;
& + .mx_BaseAvatar {
clear: both;
}
} }
.mx_SpaceRoomDirectory_list { .mx_SpaceRoomDirectory_list {

View file

@ -104,61 +104,23 @@ $spacePanelWidth: 71px;
margin-top: 18px; margin-top: 18px;
margin-bottom: 12px; margin-bottom: 12px;
> hr {
border: none;
border-top: 1px solid $input-border-color;
margin-bottom: 12px;
}
> div {
display: flex;
flex-direction: row;
font-size: $font-15px; font-size: $font-15px;
line-height: $font-24px; line-height: $font-24px;
> span { > span {
color: $secondary-fg-color; color: $secondary-fg-color;
position: relative; position: relative;
padding-left: 32px;
font-size: inherit; font-size: inherit;
line-height: inherit; line-height: inherit;
margin-right: auto; margin-right: auto;
&::before {
content: '';
position: absolute;
left: 0;
top: 2px;
height: 20px;
width: 20px;
background-color: $secondary-fg-color;
mask-repeat: no-repeat;
mask-size: contain;
mask-image: url('$(res)/img/element-icons/room/room-summary.svg');
mask-position: center;
}
} }
.mx_AccessibleButton_kind_link { .mx_AccessibleButton_kind_link {
color: $accent-color; color: $accent-color;
position: relative; position: relative;
padding: 0 0 0 24px; padding: 0;
margin-left: 8px; margin-left: 8px;
font-size: inherit; font-size: inherit;
line-height: inherit; line-height: inherit;
&::before {
content: '';
position: absolute;
left: 0;
height: 16px;
width: 16px;
background-color: $accent-color;
mask-repeat: no-repeat;
mask-size: contain;
mask-image: url('$(res)/img/element-icons/chat-bubbles.svg');
mask-position: center;
}
}
} }
} }

View file

@ -62,9 +62,8 @@ import IconizedContextMenu, {
import AccessibleTooltipButton from "../views/elements/AccessibleTooltipButton"; import AccessibleTooltipButton from "../views/elements/AccessibleTooltipButton";
import { BetaPill } from "../views/beta/BetaCard"; import { BetaPill } from "../views/beta/BetaCard";
import { UserTab } from "../views/dialogs/UserSettingsDialog"; import { UserTab } from "../views/dialogs/UserSettingsDialog";
import Modal from "../../Modal";
import BetaFeedbackDialog from "../views/dialogs/BetaFeedbackDialog";
import { EffectiveMembership, getEffectiveMembership } from "../../utils/membership"; import { EffectiveMembership, getEffectiveMembership } from "../../utils/membership";
import { SpaceFeedbackPrompt } from "../views/spaces/SpaceCreateMenu";
interface IProps { interface IProps {
space: Room; space: Room;
@ -393,19 +392,7 @@ const SpaceLanding = ({ space }) => {
}; };
return <div className="mx_SpaceRoomView_landing"> return <div className="mx_SpaceRoomView_landing">
<div className="mx_SpaceFeedbackPrompt_topRight"> <SpaceFeedbackPrompt />
{ _t("Spaces are a new feature.") }&nbsp;
<AccessibleButton
kind="link"
onClick={() => {
Modal.createTrackedDialog("Beta Feedback", "feature_spaces", BetaFeedbackDialog, {
featureId: "feature_spaces",
});
}}
>
{ _t("Give feedback.") }
</AccessibleButton>
</div>
<RoomAvatar room={space} height={80} width={80} viewAvatarOnClick={true} /> <RoomAvatar room={space} height={80} width={80} viewAvatarOnClick={true} />
<div className="mx_SpaceRoomView_landing_name"> <div className="mx_SpaceRoomView_landing_name">
<RoomName room={space}> <RoomName room={space}>

View file

@ -66,12 +66,10 @@ const nameToAlias = (name: string, domain: string): string => {
}; };
// XXX: Temporary for the Spaces release only // XXX: Temporary for the Spaces release only
const SpaceFeedbackPrompt = ({ onClick }: { onClick?: () => void }) => { export const SpaceFeedbackPrompt = ({ onClick }: { onClick?: () => void }) => {
if (!SdkConfig.get().bug_report_endpoint_url) return null; if (!SdkConfig.get().bug_report_endpoint_url) return null;
return <div className="mx_SpaceFeedbackPrompt"> return <div className="mx_SpaceFeedbackPrompt">
<hr />
<div>
<span className="mx_SpaceFeedbackPrompt_text">{ _t("Spaces are a new feature.") }</span> <span className="mx_SpaceFeedbackPrompt_text">{ _t("Spaces are a new feature.") }</span>
<AccessibleButton <AccessibleButton
kind="link" kind="link"
@ -92,7 +90,6 @@ const SpaceFeedbackPrompt = ({ onClick }: { onClick?: () => void }) => {
> >
{ _t("Give feedback.") } { _t("Give feedback.") }
</AccessibleButton> </AccessibleButton>
</div>
</div>; </div>;
}; };