Skip to:

Opened 11 years ago

Closed 11 years ago

#5240 closed enhancement (fixed)

Strings need to be changed on "Settings > Notifications" screen

Reported by: imath's profile imath Owned by: boonebgorges's profile boonebgorges
Milestone: 1.9 Priority: normal
Severity: normal Version:
Component: Settings Keywords: 2nd-opinion dev-feedback


I was playing with the great new Notifications (Screen) component, and when i've activated my settings tab i thought having a Nav and a Sub Nav with the same word might be a bit confusing. So i wonder if the Settings sub nav "Notifications" should be renamed like "Email notices" to "mark" the difference with the Notifications component.

As a french, i think we call the fact of receiving an email to inform us is "Alertes email" but it might be too strong in english "Email alerts"..

I've attached a screenshot to illustrate.

What do you think of renaming the Settings email notifications subnav ?

Attachments (1)

notifications.png (113.9 KB) - added by imath 11 years ago.

Download all attachments as: .zip

Change History (6)

11 years ago

#1 @henrywright
11 years ago

Hi imath, I was thinking the same thing. After the new Notifications component goes live, you will have some confusion over the naming of BP notifications and email notifications. I agree a distinction should be made.

#2 @r-a-y
11 years ago

  • Keywords dev-feedback added
  • Milestone changed from Awaiting Review to 2.0
  • Summary changed from New component Notifications (Screen) and Notifications (Email) of the Settings Component to Strings need to be changed on "Settings > Notifications" screen
  • Version 1.8.1 deleted

Good catch, imath!

I totally agree. I think we can change the heading of the "Settings > Notifications" screen to "Email", but keep the "notifications" slug for people who might be hardcoding the slug in their themes / plugins.

#3 @r-a-y
11 years ago

  • Milestone changed from 2.0 to 1.9

Oops, meant to put this in 1.9.

#4 @boonebgorges
11 years ago

Yup, what r-a-y said. 'Email' is good for the tab, but we can't change the 'notifications' slug for backward compatibility reasons.

#5 @boonebgorges
11 years ago

  • Owner set to boonebgorges
  • Resolution set to fixed
  • Status changed from new to closed

In 7565:

Remove 'notifications' language in Settings component

We previously used the word "notification" to refer to the email notices
sent when certain events happened on the site. This causes some confusion with
the new top-level Notifications item, so we change the wording a bit.

Fixes #5240

Props imath, r-a-y

Note: See TracTickets for help on using tickets.