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

Implement "document_title_parts" filter to apply CiviCRM title on basepage #158

Merged
merged 1 commit into from
Jun 12, 2019

Conversation

christianwach
Copy link
Member

Overview

As reported here when Clean URLs are not implemented, the <title> element is not correctly set on the base page.

Before

When Clean URLs are not implemented, the <title> element is not correctly set on the base page.

After

When Clean URLs are not implemented, the <title> element is correctly set on the base page.

Technical Details

Uses the document_title_parts filter - however, its success depends on whether the theme has implemented that method for generating the title or not.

@kcristiano
Copy link
Member

Also tested as noted in SE

@kcristiano kcristiano merged commit 09849d1 into civicrm:master Jun 12, 2019
@christianwach christianwach deleted the title-parts branch June 13, 2019 15:56
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.

2 participants