Problem/Motivation

Sites using this module in conjunction with smart_date_recur may want to use different text prefixing the Add to Calendar links for the output that shows event recurrence rule and the individual event occurrence list. For example, for the links that include the recurrence rule, text such as "Add all occurrences to Calendar" or "Add recurring event to Calendar" may be preferred, while text for the individual occurrences such as "Add individual instance to Calendar" may be preferred.

An ideal way to achieve this would be to make the "Add to Calendar" text currently hardcoded in addtocal-links.html.twig a configuration option in the plugin.

Since the provides separate configuration options already for the recurring event and the instance display, this should only involve adding a single configuration option for the prefix text.

Comments

mark_fullmer created an issue. See original summary.

mark_fullmer’s picture

Assigned: mark_fullmer » Unassigned
Status: Active » Needs review
StatusFileSize
new2.24 KB

The attached patch makes the label prefix into a configuration option.

mandclu’s picture

Thanks for your work on this, it's a great idea. One question, and maybe this is a matter of semantics: Should this be called a "label prefix" or just a "label"?

mark_fullmer’s picture

Should this be called a "label prefix" or just a "label"?

Good point. Probably just "label" will make more sense to folks. I think I labelled(!) it "label prefix" initially under the thinking that the labels were the links themselves (Outlook, Google, iCal).

  • mandclu committed d80c219 on 1.0.x authored by mark_fullmer
    Issue #3229113 by mark_fullmer, mandclu: Ability to set different label...
mandclu’s picture

Status: Needs review » Fixed

I merged this in with a few tweaks. I'll try to get a new release out shortly.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.