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

Add back in body classes for themeing. #68

Merged
merged 1 commit into from
Dec 3, 2021
Merged

Add back in body classes for themeing. #68

merged 1 commit into from
Dec 3, 2021

Conversation

homotechsual
Copy link
Contributor

This adds back in the page-civicrm style body classes missing from Drupal 8 and Drupal 9 but present in Drupal 7.

This allows themes that rely on those classes like Shoreditch to maintain functionality on Drupal 8 and 9 much easier.

This adds back in the `page-civicrm` style body classes missing from Drupal 8 and Drupal 9 but present in Drupal 7.

This allows themes that rely on those classes like Shoreditch to maintain functionality on Drupal 8 and 9 much easier.
@mlutfy
Copy link
Member

mlutfy commented Nov 26, 2021

@sunilpawar As someone who has been pretty active in these areas, any thoughts on this PR?

@sunilpawar
Copy link
Contributor

@mlutfy tested functionality using Bartik theme for CiviCRM Administration theme.
Shoreditch extension was enabled.

Before : Vertical tab not present
After : Vertical tab present.

@mlutfy
Copy link
Member

mlutfy commented Nov 26, 2021

Thanks @sunilpawar !

I will leave open a few days in case anyone has objections. It was also discussed on the chat: https://chat.civicrm.org/civicrm/pl/j19dch5qhbgi7egckp9xs3hswo (Drupal channel)

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.

4 participants