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

Fix page layout for wordpress (dev/core/issues/21) #11827

Merged
merged 1 commit into from
Mar 19, 2018

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Mar 19, 2018

Overview

Reverse formatting change that caused a regression in 4.7.31 for the display of contribution pages in wordpress

Before

screenshot 2018-03-19 14 19 59

After

screenshot 2018-03-19 14 30 49

Technical Details

Change reverses a css change in 036aa39#diff-544215f9c296467a07bab9c92c448be0R71

Comments

I think this will cause something in PR #11396 to regress but that can be fixed in a more leisurely fashion later - this appears as a serious regression that needs to be fixed in the next code drop

@eileenmcnaughton
Copy link
Contributor Author

@colemanw one for you

@totten
Copy link
Member

totten commented Mar 19, 2018

jenkins, test this please

@eileenmcnaughton eileenmcnaughton changed the title Fix page layout for wordpress Fix page layout for wordpress (dev/core/issues/21) Mar 19, 2018
@colemanw
Copy link
Member

FYI @mukeshcompucorp I'm merging this due to the regression.

@colemanw colemanw merged commit 7713e22 into civicrm:5.0 Mar 19, 2018
@colemanw colemanw deleted the fix_wp branch March 19, 2018 06:16
@mukeshcompucorp
Copy link
Contributor

@colemanw sorry, I didn't got time to work on this. Yes, it seems to merge it safely.

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

Successfully merging this pull request may close these issues.

5 participants