fix: changed overlay position and updated notification info text (ope…

…nedx#849)
Abstract-Tech · Jan 9, 2024 · cd28018 · cd28018
1 parent 6327f3d
commit cd28018
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/src/notification-preferences/NotificationPreferenceRow.jsx b/src/notification-preferences/NotificationPreferenceRow.jsx
@@ -45,7 +45,7 @@ const NotificationPreferenceRow = ({ appId, preferenceName }) => {
           <OverlayTrigger
             id={tooltipId}
             className="d-inline"
-            placement="top"
+            placement="right"
             overlay={(
               <Tooltip id={tooltipId}>
                 {preference.info}

diff --git a/src/notification-preferences/messages.js b/src/notification-preferences/messages.js
@@ -21,14 +21,9 @@ export const messages = defineMessages({
     id: 'notification.preference.title',
     defaultMessage: `{
         text, select,
-        core {Core}
-        newPost {New Post}
-        newComment {New Comment}
-        newResponse {New Response}
-        newCommentOnPost {New Comment On Post}
-        newResponseOnComment {New Response On Comment}
-        newAssignment {New Assignment}
-        newGrade {New Grade}
+        core {Core notifications}
+        newDiscussionPost {New discussion posts}
+        newQuestionPost {New question posts}
         other {{text}}
     }`,
     description: 'Display text for Notification Types',