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 apparent typo, align_center to aligncenter #22 #23

Open
wants to merge 1 commit into
base: feature/site-plugin-list-block
Choose a base branch
from

Conversation

presswizards
Copy link

@presswizards presswizards commented Mar 17, 2023

#22

What is the desired behavior?

Using shortcode:
[wp-pic type="plugin" slug="pretty-simple-popup-builder" layout="large" scheme="scheme2" align="center" ajax="yes" ]

It should center the large layout.

What is the actual behavior?

It does not center, the CSS I see is:
class="wp-pic-wrapper align_center large"

align_center is not in the plugin's CSS. I do see .wp-pic-wrapper.aligncenter.large and .wp-pic-wrapper.alignfull.large etc in the CSS, but it's not applied via the shortcode it seems.

Steps to reproduce undesired behavior:

Try using the shortcode with align="center".

Screenshots or Video

I've added custom CSS to resolve the issue:

.wp-pic-wrapper.align_center {
text-align: center;
}

You can see my live page here, centered because of the above CSS, uncheck that CSS to view it not centered:
https://5starplugins.com/pretty-simple-popup-builder/

Additional Comments

Hope you can fix it, I'm sure it's affecting other sites.

@presswizards presswizards changed the title Fix apparent typo, align_center to aligncenter Fix apparent typo, align_center to aligncenter #22 Mar 17, 2023
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.

1 participant