CiviMember: use fa-ban
, proper crm-i
class for canceled auto-renew
#11775
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
In CRM-21688, icons display the status of auto-renew for memberships. This uses the proper class for CiviCRM's implementation of Font Awesome and changes the icon for canceled auto-renewals.
Before
The icons use the
fa
class, which is the standard class for Font Awesome icons. However, this could result in problems with CMS themes that have their own implementations of Font Awesome.Also, the icon for canceled auto-renewals is
fa-exclamation
, which implies that something has gone wrong with the autorenewal and needs immediate attention.After
The icons use the
crm-i
class, which explicitly uses CiviCRM's Font Awesome library.The icon for canceled auto-renewals is
fa-ban
, an icon that more closely expresses that the member has canceled their autorenewal.Comments
I'm not sure, but I don't think that CiviCRM's Font Awesome responds to theNever mind, I just checked and it appears okay on dmaster.fa
class at all, which means that the icons would be missing if a site lacks a separate Font Awesome implementation. @colemanw do you know? If that's the case, I think this might need to be a last-minute merge to the 4.7.31 RC.