diff --git a/res/css/views/settings/tabs/_SettingsBanner.pcss b/res/css/views/settings/tabs/_SettingsBanner.pcss index f0b9e7106bb..a6a0de63459 100644 --- a/res/css/views/settings/tabs/_SettingsBanner.pcss +++ b/res/css/views/settings/tabs/_SettingsBanner.pcss @@ -16,7 +16,6 @@ limitations under the License. .mx_SettingsBanner { background: $system; - line-height: 2.25rem; border-radius: 8px; padding: 12px 16px; gap: 12px; @@ -29,7 +28,6 @@ limitations under the License. } .mx_AccessibleButton { - align-self: initial; white-space: nowrap; } } diff --git a/src/components/views/settings/notifications/NotificationSettings2.tsx b/src/components/views/settings/notifications/NotificationSettings2.tsx index c3ba045602b..56b4d7acd7e 100644 --- a/src/components/views/settings/notifications/NotificationSettings2.tsx +++ b/src/components/views/settings/notifications/NotificationSettings2.tsx @@ -31,6 +31,7 @@ import SettingsStore from "../../../../settings/SettingsStore"; import { NotificationColor } from "../../../../stores/notifications/NotificationColor"; import { clearAllNotifications } from "../../../../utils/notifications"; import AccessibleButton from "../../elements/AccessibleButton"; +import ExternalLink from "../../elements/ExternalLink"; import LabelledCheckbox from "../../elements/LabelledCheckbox"; import LabelledToggleSwitch from "../../elements/LabelledToggleSwitch"; import StyledRadioGroup from "../../elements/StyledRadioGroup"; @@ -76,6 +77,10 @@ function boldText(text: string): JSX.Element { return {text}; } +function helpLink(sub: string): JSX.Element { + return {sub}; +} + function useHasUnreadNotifications(): boolean { const cli = useMatrixClientContext(); return cli.getRooms().some((room) => room.getUnreadNotificationCount() > 0); @@ -101,13 +106,20 @@ export default function NotificationSettings2(): JSX.Element { {hasPendingChanges && model !== null && ( } - action={_t("Switch now")} + action={_t("Proceed")} onAction={() => reconcile(model!)} > {_t( - "Update: We have updated our notification settings. This won’t affect your previously selected settings.", + "Update:" + + "We’ve simplified Notifications Settings to make options easier to find. " + + "Some custom settings you’ve chosen in the past are not shown here, but they’re still active. " + + "If you proceed, some of your settings may change. " + + "Learn more", {}, - { strong: boldText }, + { + strong: boldText, + a: helpLink, + }, )} )} diff --git a/src/i18n/strings/en_EN.json b/src/i18n/strings/en_EN.json index b545df98923..63c29e9f65e 100644 --- a/src/i18n/strings/en_EN.json +++ b/src/i18n/strings/en_EN.json @@ -1782,8 +1782,8 @@ "Select which emails you want to send summaries to. Manage your emails in .": "Select which emails you want to send summaries to. Manage your emails in .", "People, Mentions and Keywords": "People, Mentions and Keywords", "Mentions and Keywords only": "Mentions and Keywords only", - "Switch now": "Switch now", - "Update: We have updated our notification settings. This won’t affect your previously selected settings.": "Update: We have updated our notification settings. This won’t affect your previously selected settings.", + "Proceed": "Proceed", + "Update:We’ve simplified Notifications Settings to make options easier to find. Some custom settings you’ve chosen in the past are not shown here, but they’re still active. If you proceed, some of your settings may change. Learn more": "Update:We’ve simplified Notifications Settings to make options easier to find. Some custom settings you’ve chosen in the past are not shown here, but they’re still active. If you proceed, some of your settings may change. Learn more", "Show message preview in desktop notification": "Show message preview in desktop notification", "I want to be notified for (Default Setting)": "I want to be notified for (Default Setting)", "This setting will be applied by default to all your rooms.": "This setting will be applied by default to all your rooms.",