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

Create real menu item so the position, icon & label can be changed. #9

Merged
merged 1 commit into from
Apr 28, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Apr 6, 2022

Uses hook_civicrm_managed to insert a real navigation item, which serves as the base for the recent items menu. It can be repositioned anywhere in the navigation tree, even nested inside another menu.

image

Uses hook_civicrm_managed to insert a real navigation item,
which serves as the base for the recent items menu. It can be repositioned
anywhere in the navigation tree, even nested inside another menu.
@colemanw
Copy link
Member Author

@bjendres have you had a chance to test this?

@bjendres
Copy link

@bjendres have you had a chance to test this?

No, sorry, the client who needs this is on an older CiviCRM version, so that doesn't work for them.

@colemanw
Copy link
Member Author

What version are they using? You could apply this patch to their extension and it would work for CiviCRM 5.45+

@bjendres
Copy link

What version are they using? You could apply this patch to their extension and it would work for CiviCRM 5.45+

That's really nice of you, but
a) it's 5.37.2
b) they've already settled of the original version now

Thanks a lot anyway.

@colemanw colemanw merged commit bb2a15f into master Apr 28, 2022
@colemanw colemanw deleted the managedNav branch April 28, 2022 15:16
@colemanw
Copy link
Member Author

Ok well this will get used at some point in the future.

@bjendres
Copy link

Ok well this will get used at some point in the future.

Absolutely, thanks for all your work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants