Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Afform - Allow picking icon for tab, add CrmUiIconPicker widget #23919

Merged
merged 2 commits into from
Jul 4, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jun 30, 2022

Overview

When embedding an afform in the contact summary tab, this allows you to pick an icon.
This also adds the correct icon to the Grants tab for the CiviGrant extension.

Also see: https://lab.civicrm.org/extensions/certifications/-/issues/9

Before

Icon could be customized with the Contact Layout Editor, which works for local customizations but not for packaged Afforms.

After

Icon can be specified in Afform GUI, and in packaged Afform json.

Technical Details

Since this is needed in several places, this adds a general purpose crmUiIconPicker widget to crmUi.
It also switches Afforms's hook_civicrm_tabset to use the API, which ensures virtual forms get picked up.

@civibot civibot bot added the master label Jun 30, 2022
@civibot
Copy link

civibot bot commented Jun 30, 2022

(Standard links)

colemanw added 2 commits June 30, 2022 14:50
Allows afforms placed as a contact summary tab to specify an icon.
It also switches Afforms's hook_civicrm_tabset to use the API, which ensures virtual forms get picked up.
Adds the new icon for CiviGrant as an example.
@aydun
Copy link
Contributor

aydun commented Jul 4, 2022

Looks good - that's nice having the popup icon picker rather than requiring you to know the icon name.

r-run: ok - loads, lets you pick the icon, saves it, displays on the tab. All good.

@demeritcowboy demeritcowboy merged commit 1962c05 into civicrm:master Jul 4, 2022
@colemanw colemanw deleted the afformIcon branch July 4, 2022 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants