Make WordPress Core

Opened 4 years ago

Last modified 5 months ago

#52480 new enhancement

Refine the display of the comment approval notification opt-in confirmation message

Reported by: johnbillion's profile johnbillion Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Comments Keywords: 2nd-opinion
Focuses: accessibility Cc:

Description

Follow-up to #52406.

The confirmation message shown to a user after they opt-in to receiving a notification of their pending comment's approval is currently displayed inline with a preview of their comment.

This means the display of this confirmation message is only shown if the user opts in within 10 minutes of posting their comment. If they take longer than 10 minutes then their opt-in is respected but they see no confirmation message.

As mentioned in the comments in 7:ticket:52406 10 minutes is likely long enough but no research was done.

Let's identify if this functionality needs to be improved. Options:

  • Extend the time limit further. This needs to take into consideration #49956.
  • Disconnect the display of the confirmation message from the comment preview, and always show a confirmation message. Would need to take into consideration the cache headers on the page.
  • Fix the comment spam problem some other way and remove the time limit.
  • Do something else.
  • Do nothing and leave it as-is.

Change History (9)

This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.


4 years ago

This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.


4 years ago

This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.


4 years ago

#4 @ryokuhi
4 years ago

  • Milestone changed from Awaiting Review to 5.8

Moving it to 5.8 after discussion during the Accessibility Team's bugscrub.

#5 @johnbillion
4 years ago

  • Milestone changed from 5.8 to Awaiting Review
  • Version 5.7 deleted

This is dependent on #33717 going back in.

This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.


4 years ago

This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.


4 years ago

#8 @joedolson
4 years ago

  • Milestone changed from Awaiting Review to Future Release

Moving this to Future Release. Need to monitor #33717 and follow up if it is re-committed, but don't need to keep reviewing it every week...

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


5 months ago

Note: See TracTickets for help on using tickets.