Skip to:

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#7646 closed enhancement (fixed)

Email text link colors should be separately configurable

Reported by: boonebgorges's profile boonebgorges Owned by: djpaul's profile DJPaul
Milestone: 3.0 Priority: normal
Severity: normal Version:
Component: Emails Keywords: needs-patch good-first-bug


Currently, email text link color is set to highlight_color. Because of the weird dance of bp_email_add_link_color_to_template(), the only way to make the link color something other than highlight_color is to hook your own version of bp_email_add_link_color_to_template().

I suggest having link_color be a separate item that can be configured by filtering the 'email_appearance_settings' bp_parse_args() hooks. If link_color is not set explicitly, it can fall back on highlight_color.

Change History (2)

#1 @djpaul
6 years ago

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

In 11891:

Emails: add support for changing email text link colour.

Adds a link_text_color option to bp_email_get_appearance_settings(),
which allows a developer to filter the email text link colour. This new
colour option is not exposed in the Customiser; that continues to use the
"highlight" colour for email text links.

Fixes #7646

#2 @DJPaul
6 years ago

  • Milestone changed from Awaiting Contributions to 3.0
Note: See TracTickets for help on using tickets.