mirror of https://github.com/vector-im/riot-web
Remove threads beta feedback button (#10038)
parent
16addd6ead
commit
8161da1054
|
@ -1,5 +1,5 @@
|
||||||
/*
|
/*
|
||||||
Copyright 2021 - 2022 The Matrix.org Foundation C.I.C.
|
Copyright 2021 - 2023 The Matrix.org Foundation C.I.C.
|
||||||
|
|
||||||
Licensed under the Apache License, Version 2.0 (the "License");
|
Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
you may not use this file except in compliance with the License.
|
you may not use this file except in compliance with the License.
|
||||||
|
@ -32,16 +32,9 @@ import { Layout } from "../../settings/enums/Layout";
|
||||||
import { RoomPermalinkCreator } from "../../utils/permalinks/Permalinks";
|
import { RoomPermalinkCreator } from "../../utils/permalinks/Permalinks";
|
||||||
import Measured from "../views/elements/Measured";
|
import Measured from "../views/elements/Measured";
|
||||||
import PosthogTrackers from "../../PosthogTrackers";
|
import PosthogTrackers from "../../PosthogTrackers";
|
||||||
import AccessibleButton, { ButtonEvent } from "../views/elements/AccessibleButton";
|
import { ButtonEvent } from "../views/elements/AccessibleButton";
|
||||||
import { BetaPill } from "../views/beta/BetaCard";
|
|
||||||
import Modal from "../../Modal";
|
|
||||||
import BetaFeedbackDialog from "../views/dialogs/BetaFeedbackDialog";
|
|
||||||
import { Action } from "../../dispatcher/actions";
|
|
||||||
import { UserTab } from "../views/dialogs/UserTab";
|
|
||||||
import dis from "../../dispatcher/dispatcher";
|
|
||||||
import Spinner from "../views/elements/Spinner";
|
import Spinner from "../views/elements/Spinner";
|
||||||
import Heading from "../views/typography/Heading";
|
import Heading from "../views/typography/Heading";
|
||||||
import { shouldShowFeedback } from "../../utils/Feedback";
|
|
||||||
|
|
||||||
interface IProps {
|
interface IProps {
|
||||||
roomId: string;
|
roomId: string;
|
||||||
|
@ -231,14 +224,6 @@ const ThreadPanel: React.FC<IProps> = ({ roomId, onClose, permalinkCreator }) =>
|
||||||
}
|
}
|
||||||
}, [timelineSet, timelinePanel]);
|
}, [timelineSet, timelinePanel]);
|
||||||
|
|
||||||
const openFeedback = shouldShowFeedback()
|
|
||||||
? () => {
|
|
||||||
Modal.createDialog(BetaFeedbackDialog, {
|
|
||||||
featureId: "feature_threadenabled",
|
|
||||||
});
|
|
||||||
}
|
|
||||||
: null;
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<RoomContext.Provider
|
<RoomContext.Provider
|
||||||
value={{
|
value={{
|
||||||
|
@ -256,32 +241,6 @@ const ThreadPanel: React.FC<IProps> = ({ roomId, onClose, permalinkCreator }) =>
|
||||||
empty={!hasThreads}
|
empty={!hasThreads}
|
||||||
/>
|
/>
|
||||||
}
|
}
|
||||||
footer={
|
|
||||||
<>
|
|
||||||
<BetaPill
|
|
||||||
tooltipTitle={_t("Threads are a beta feature")}
|
|
||||||
tooltipCaption={_t("Click for more info")}
|
|
||||||
onClick={() => {
|
|
||||||
dis.dispatch({
|
|
||||||
action: Action.ViewUserSettings,
|
|
||||||
initialTabId: UserTab.Labs,
|
|
||||||
});
|
|
||||||
}}
|
|
||||||
/>
|
|
||||||
{openFeedback &&
|
|
||||||
_t(
|
|
||||||
"<a>Give feedback</a>",
|
|
||||||
{},
|
|
||||||
{
|
|
||||||
a: (sub) => (
|
|
||||||
<AccessibleButton kind="link_inline" onClick={openFeedback}>
|
|
||||||
{sub}
|
|
||||||
</AccessibleButton>
|
|
||||||
),
|
|
||||||
},
|
|
||||||
)}
|
|
||||||
</>
|
|
||||||
}
|
|
||||||
className="mx_ThreadPanel"
|
className="mx_ThreadPanel"
|
||||||
onClose={onClose}
|
onClose={onClose}
|
||||||
withoutScrollContainer={true}
|
withoutScrollContainer={true}
|
||||||
|
|
|
@ -3449,8 +3449,6 @@
|
||||||
"Threads help keep your conversations on-topic and easy to track.": "Threads help keep your conversations on-topic and easy to track.",
|
"Threads help keep your conversations on-topic and easy to track.": "Threads help keep your conversations on-topic and easy to track.",
|
||||||
"<b>Tip:</b> Use “%(replyInThread)s” when hovering over a message.": "<b>Tip:</b> Use “%(replyInThread)s” when hovering over a message.",
|
"<b>Tip:</b> Use “%(replyInThread)s” when hovering over a message.": "<b>Tip:</b> Use “%(replyInThread)s” when hovering over a message.",
|
||||||
"Keep discussions organised with threads": "Keep discussions organised with threads",
|
"Keep discussions organised with threads": "Keep discussions organised with threads",
|
||||||
"Threads are a beta feature": "Threads are a beta feature",
|
|
||||||
"<a>Give feedback</a>": "<a>Give feedback</a>",
|
|
||||||
"Thread": "Thread",
|
"Thread": "Thread",
|
||||||
"Tried to load a specific point in this room's timeline, but you do not have permission to view the message in question.": "Tried to load a specific point in this room's timeline, but you do not have permission to view the message in question.",
|
"Tried to load a specific point in this room's timeline, but you do not have permission to view the message in question.": "Tried to load a specific point in this room's timeline, but you do not have permission to view the message in question.",
|
||||||
"Tried to load a specific point in this room's timeline, but was unable to find it.": "Tried to load a specific point in this room's timeline, but was unable to find it.",
|
"Tried to load a specific point in this room's timeline, but was unable to find it.": "Tried to load a specific point in this room's timeline, but was unable to find it.",
|
||||||
|
|
|
@ -26,53 +26,14 @@ import MatrixClientContext from "../../../src/contexts/MatrixClientContext";
|
||||||
import RoomContext from "../../../src/contexts/RoomContext";
|
import RoomContext from "../../../src/contexts/RoomContext";
|
||||||
import { _t } from "../../../src/languageHandler";
|
import { _t } from "../../../src/languageHandler";
|
||||||
import { MatrixClientPeg } from "../../../src/MatrixClientPeg";
|
import { MatrixClientPeg } from "../../../src/MatrixClientPeg";
|
||||||
import { shouldShowFeedback } from "../../../src/utils/Feedback";
|
|
||||||
import { RoomPermalinkCreator } from "../../../src/utils/permalinks/Permalinks";
|
import { RoomPermalinkCreator } from "../../../src/utils/permalinks/Permalinks";
|
||||||
import ResizeNotifier from "../../../src/utils/ResizeNotifier";
|
import ResizeNotifier from "../../../src/utils/ResizeNotifier";
|
||||||
import { createTestClient, getRoomContext, mkStubRoom, mockPlatformPeg, stubClient } from "../../test-utils";
|
import { getRoomContext, mockPlatformPeg, stubClient } from "../../test-utils";
|
||||||
import { mkThread } from "../../test-utils/threads";
|
import { mkThread } from "../../test-utils/threads";
|
||||||
|
|
||||||
jest.mock("../../../src/utils/Feedback");
|
jest.mock("../../../src/utils/Feedback");
|
||||||
|
|
||||||
describe("ThreadPanel", () => {
|
describe("ThreadPanel", () => {
|
||||||
describe("Feedback prompt", () => {
|
|
||||||
const cli = createTestClient();
|
|
||||||
const room = mkStubRoom("!room:server", "room", cli);
|
|
||||||
mocked(cli.getRoom).mockReturnValue(room);
|
|
||||||
|
|
||||||
it("should show feedback prompt if feedback is enabled", () => {
|
|
||||||
mocked(shouldShowFeedback).mockReturnValue(true);
|
|
||||||
|
|
||||||
render(
|
|
||||||
<MatrixClientContext.Provider value={cli}>
|
|
||||||
<ThreadPanel
|
|
||||||
roomId="!room:server"
|
|
||||||
onClose={jest.fn()}
|
|
||||||
resizeNotifier={new ResizeNotifier()}
|
|
||||||
permalinkCreator={new RoomPermalinkCreator(room)}
|
|
||||||
/>
|
|
||||||
</MatrixClientContext.Provider>,
|
|
||||||
);
|
|
||||||
expect(screen.queryByText("Give feedback")).toBeTruthy();
|
|
||||||
});
|
|
||||||
|
|
||||||
it("should hide feedback prompt if feedback is disabled", () => {
|
|
||||||
mocked(shouldShowFeedback).mockReturnValue(false);
|
|
||||||
|
|
||||||
render(
|
|
||||||
<MatrixClientContext.Provider value={cli}>
|
|
||||||
<ThreadPanel
|
|
||||||
roomId="!room:server"
|
|
||||||
onClose={jest.fn()}
|
|
||||||
resizeNotifier={new ResizeNotifier()}
|
|
||||||
permalinkCreator={new RoomPermalinkCreator(room)}
|
|
||||||
/>
|
|
||||||
</MatrixClientContext.Provider>,
|
|
||||||
);
|
|
||||||
expect(screen.queryByText("Give feedback")).toBeFalsy();
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe("Header", () => {
|
describe("Header", () => {
|
||||||
it("expect that All filter for ThreadPanelHeader properly renders Show: All threads", () => {
|
it("expect that All filter for ThreadPanelHeader properly renders Show: All threads", () => {
|
||||||
const { asFragment } = render(
|
const { asFragment } = render(
|
||||||
|
|
Loading…
Reference in New Issue