diff --git a/package.json b/package.json index ad38d8f197d..68f7654a93d 100644 --- a/package.json +++ b/package.json @@ -91,7 +91,7 @@ "linkifyjs": "^4.0.0-beta.4", "lodash": "^4.17.20", "maplibre-gl": "^1.15.2", - "matrix-analytics-events": "github:matrix-org/matrix-analytics-events.git#8e75aaf0b3e045587daeaf97a7691dbfda2f20c0", + "matrix-analytics-events": "github:matrix-org/matrix-analytics-events.git#daad3faed54f0b1f1e026a7498b4653e4d01cd90", "matrix-events-sdk": "^0.0.1-beta.7", "matrix-js-sdk": "github:matrix-org/matrix-js-sdk#develop", "matrix-widget-api": "^0.1.0-beta.18", diff --git a/src/PosthogAnalytics.ts b/src/PosthogAnalytics.ts index 343aea62358..3b2ae009aec 100644 --- a/src/PosthogAnalytics.ts +++ b/src/PosthogAnalytics.ts @@ -152,13 +152,13 @@ export class PosthogAnalytics { // we persist the last `$screen_name` and send it for all events until it is replaced private lastScreen: ScreenName = "Loading"; - private sanitizeProperties = (properties: posthog.Properties): posthog.Properties => { + private sanitizeProperties = (properties: posthog.Properties, eventName: string): posthog.Properties => { // Callback from posthog to sanitize properties before sending them to the server. // // Here we sanitize posthog's built in properties which leak PII e.g. url reporting. // See utils.js _.info.properties in posthog-js. - if (properties["eventName"] === "$pageview") { + if (eventName === "$pageview") { this.lastScreen = properties["$current_url"]; } // We inject a screen identifier in $current_url as per https://posthog.com/tutorials/spa diff --git a/src/components/structures/ThreadPanel.tsx b/src/components/structures/ThreadPanel.tsx index ec8d4c9caed..fe0228b13b9 100644 --- a/src/components/structures/ThreadPanel.tsx +++ b/src/components/structures/ThreadPanel.tsx @@ -38,6 +38,8 @@ import TimelinePanel from './TimelinePanel'; import { Layout } from '../../settings/enums/Layout'; import { RoomPermalinkCreator } from '../../utils/permalinks/Permalinks'; import Measured from '../views/elements/Measured'; +import PosthogTrackers from "../../PosthogTrackers"; +import { ButtonEvent } from "../views/elements/AccessibleButton"; export async function getThreadTimelineSet( client: MatrixClient, @@ -178,7 +180,15 @@ export const ThreadPanelHeader = ({ filterOption, setFilterOption, empty }: { return
{ _t("Threads") } { !empty && <> - menuDisplayed ? closeMenu() : openMenu()}> + { + openMenu(); + PosthogTrackers.trackInteraction("WebRightPanelThreadPanelFilterDropdown", ev); + }} + > { `${_t('Show:')} ${value.label}` } { contextMenu } diff --git a/src/components/structures/ThreadView.tsx b/src/components/structures/ThreadView.tsx index 1abbf2aa494..b3b29918538 100644 --- a/src/components/structures/ThreadView.tsx +++ b/src/components/structures/ThreadView.tsx @@ -50,6 +50,8 @@ import FileDropTarget from "./FileDropTarget"; import { getKeyBindingsManager } from "../../KeyBindingsManager"; import { KeyBindingAction } from "../../accessibility/KeyboardShortcuts"; import Measured from '../views/elements/Measured'; +import PosthogTrackers from "../../PosthogTrackers"; +import { ButtonEvent } from "../views/elements/AccessibleButton"; interface IProps { room: Room; @@ -321,6 +323,9 @@ export default class ThreadView extends React.Component { header={this.renderThreadViewHeader()} ref={this.card} onKeyDown={this.onKeyDown} + onBack={(ev: ButtonEvent) => { + PosthogTrackers.trackInteraction("WebThreadViewBackButton", ev); + }} > void; + onClick: (ev: ButtonEvent) => void; // The parameters to track the click event analytics: Parameters; @@ -42,9 +43,9 @@ interface IProps { // TODO: replace this, the composer buttons and the right panel buttons with a unified representation @replaceableComponent("views.right_panel.HeaderButton") export default class HeaderButton extends React.Component { - private onClick = () => { + private onClick = (ev: ButtonEvent) => { Analytics.trackEvent(...this.props.analytics); - this.props.onClick(); + this.props.onClick(ev); }; public render() { diff --git a/src/components/views/right_panel/RoomHeaderButtons.tsx b/src/components/views/right_panel/RoomHeaderButtons.tsx index 6d4ddb7b589..acca6f935c6 100644 --- a/src/components/views/right_panel/RoomHeaderButtons.tsx +++ b/src/components/views/right_panel/RoomHeaderButtons.tsx @@ -38,6 +38,8 @@ import { RoomNotificationStateStore } from "../../../stores/notifications/RoomNo import { NotificationColor } from "../../../stores/notifications/NotificationColor"; import { ThreadsRoomNotificationState } from "../../../stores/notifications/ThreadsRoomNotificationState"; import { NotificationStateEvents } from "../../../stores/notifications/NotificationState"; +import PosthogTrackers from "../../../PosthogTrackers"; +import { ButtonEvent } from "../elements/AccessibleButton"; const ROOM_INFO_PHASES = [ RightPanelPhases.RoomSummary, @@ -207,11 +209,12 @@ export default class RoomHeaderButtons extends HeaderButtons { this.setPhase(RightPanelPhases.Timeline); }; - private onThreadsPanelClicked = () => { + private onThreadsPanelClicked = (ev: ButtonEvent) => { if (RoomHeaderButtons.THREAD_PHASES.includes(this.state.phase)) { RightPanelStore.instance.togglePanel(); } else { showThreadPanel(); + PosthogTrackers.trackInteraction("WebRoomHeaderButtonsThreadsButton", ev); } }; diff --git a/src/components/views/rooms/EventTile.tsx b/src/components/views/rooms/EventTile.tsx index dbd4e8ee985..fa2ef79829d 100644 --- a/src/components/views/rooms/EventTile.tsx +++ b/src/components/views/rooms/EventTile.tsx @@ -15,7 +15,7 @@ See the License for the specific language governing permissions and limitations under the License. */ -import React, { createRef } from 'react'; +import React, { createRef, MouseEvent } from 'react'; import classNames from "classnames"; import { EventType, MsgType, RelationType } from "matrix-js-sdk/src/@types/event"; import { EventStatus, MatrixEvent, MatrixEventEvent } from "matrix-js-sdk/src/models/event"; @@ -81,6 +81,7 @@ import { DecryptionFailureTracker } from '../../../DecryptionFailureTracker'; import RedactedBody from '../messages/RedactedBody'; import { ViewRoomPayload } from "../../../dispatcher/payloads/ViewRoomPayload"; import { shouldDisplayReply } from '../../../utils/Reply'; +import PosthogTrackers from "../../../PosthogTrackers"; export type GetRelationsForEvent = (eventId: string, relationType: string, eventType: string) => Relations; @@ -684,17 +685,19 @@ export default class EventTile extends React.Component { return ( { context => -
{ + onClick={(ev: ButtonEvent) => { showThread({ rootEvent: this.props.mxEvent, push: context.isCard }); + PosthogTrackers.trackInteraction("WebRoomTimelineThreadSummaryButton", ev); }} + aria-label={_t("Open thread")} > { count } { this.renderThreadLastMessagePreview() } -
+ } ); @@ -1507,7 +1510,12 @@ export default class EventTile extends React.Component { "data-notification": this.state.threadNotification, "onMouseEnter": () => this.setState({ hover: true }), "onMouseLeave": () => this.setState({ hover: false }), - "onClick": () => showThread({ rootEvent: this.props.mxEvent, push: true }), + "onClick": (ev: MouseEvent) => { + showThread({ rootEvent: this.props.mxEvent, push: true }); + const target = ev.currentTarget as HTMLElement; + const index = Array.from(target.parentElement.children).indexOf(target); + PosthogTrackers.trackInteraction("WebThreadsPanelThreadItem", ev, index); + }, }, <> { sender } { avatar } diff --git a/src/components/views/rooms/SendMessageComposer.tsx b/src/components/views/rooms/SendMessageComposer.tsx index 65b2b70f685..4274db9bf5e 100644 --- a/src/components/views/rooms/SendMessageComposer.tsx +++ b/src/components/views/rooms/SendMessageComposer.tsx @@ -345,12 +345,17 @@ export class SendMessageComposer extends React.Component({ + const posthogEvent: ComposerEvent = { eventName: "Composer", isEditing: false, - inThread: this.props.relation?.rel_type === RelationType.Thread, isReply: !!this.props.replyToEvent, - }); + inThread: this.props.relation?.rel_type === RelationType.Thread, + }; + if (posthogEvent.inThread) { + const threadRoot = this.props.room.findEventById(this.props.relation.event_id); + posthogEvent.startsThread = threadRoot?.getThread()?.events.length === 1; + } + PosthogAnalytics.instance.trackEvent(posthogEvent); // Replace emoticon at the end of the message if (SettingsStore.getValue('MessageComposerInput.autoReplaceEmoji')) { diff --git a/src/i18n/strings/en_EN.json b/src/i18n/strings/en_EN.json index ed22ff29b0b..e70c96d9840 100644 --- a/src/i18n/strings/en_EN.json +++ b/src/i18n/strings/en_EN.json @@ -1659,6 +1659,7 @@ "From a thread": "From a thread", "%(count)s reply|other": "%(count)s replies", "%(count)s reply|one": "%(count)s reply", + "Open thread": "Open thread", "This event could not be displayed": "This event could not be displayed", "Your key share request has been sent - please check your other sessions for key share requests.": "Your key share request has been sent - please check your other sessions for key share requests.", "Key share requests are sent to your other sessions automatically. If you rejected or dismissed the key share request on your other sessions, click here to request the keys for this session again.": "Key share requests are sent to your other sessions automatically. If you rejected or dismissed the key share request on your other sessions, click here to request the keys for this session again.", diff --git a/yarn.lock b/yarn.lock index 88312afa8cc..00a6ca374f6 100644 --- a/yarn.lock +++ b/yarn.lock @@ -6261,9 +6261,9 @@ mathml-tag-names@^2.1.3: resolved "https://registry.yarnpkg.com/mathml-tag-names/-/mathml-tag-names-2.1.3.tgz#4ddadd67308e780cf16a47685878ee27b736a0a3" integrity sha512-APMBEanjybaPzUrfqU0IMU5I0AswKMH7k8OTLs0vvV4KZpExkTkY87nR/zpbuTPj+gARop7aGUbl11pnDfW6xg== -"matrix-analytics-events@github:matrix-org/matrix-analytics-events.git#8e75aaf0b3e045587daeaf97a7691dbfda2f20c0": +"matrix-analytics-events@github:matrix-org/matrix-analytics-events.git#daad3faed54f0b1f1e026a7498b4653e4d01cd90": version "0.0.1" - resolved "https://codeload.github.com/matrix-org/matrix-analytics-events/tar.gz/8e75aaf0b3e045587daeaf97a7691dbfda2f20c0" + resolved "https://codeload.github.com/matrix-org/matrix-analytics-events/tar.gz/daad3faed54f0b1f1e026a7498b4653e4d01cd90" matrix-events-sdk@^0.0.1-beta.6: version "0.0.1-beta.6"