Opened 5 years ago
Closed 5 years ago
#8168 closed defect (bug) (fixed)
BuddyPress Email footer Error
Reported by: | passoniate | Owned by: | DJPaul |
---|---|---|---|
Milestone: | 6.0.0 | Priority: | normal |
Severity: | normal | Version: | 5.0.0 |
Component: | Emails | Keywords: | has-patch commit |
Cc: |
Description
When i add continuous characters in Email Footer then characters out from footer container https://paste.pics/7CPAT
Attachments (4)
Change History (13)
This ticket was mentioned in Slack in #buddypress by passoniate. View the logs.
5 years ago
#2
@
5 years ago
- Component changed from Core to Emails
- Milestone changed from Awaiting Review to 6.0.0
- Owner set to DJPaul
This ticket was mentioned in Slack in #buddypress by passoniate. View the logs.
5 years ago
#6
@
5 years ago
- Keywords has-patch 2nd-opinion added; needs-patch removed
I've looked into it more deeply. I believe we should do the minimum styling there and I'm very close to believe we shouldn't do anything at all actually.
I've added 8168.patch to simply add a word-break
rule.
I think if a plugin developer hooks to bp_before_email_footer
or any other hook into the email template or if the site owner or the BuddyPress theme developer edits the template, then it's probably best to let them control how the code they're adding looks.
I'd be interested to have a second opinion about if from another member of the team 🙏.
Hi!
Thanks a lot for your report. Let's try to work on this during the 6.0.0 development cycle.