search

INN / inn

WordPress child theme for INN.org. Requires Largo parent theme (https://github.com/INN/largo)
https://inn.org
GNU General Public License v2.0
2 stars 2 forks source link

Update 180+ count to 200+ #106

Closed benlk closed 5 years ago

benlk commented 5 years ago

Raised by @MirandaEcho in today's meeting: https://github.com/INN/inn/blob/ae9f2e1563f6c43f99cd4d01033b60f0ca2d6f6b/homepages/templates/inn.php#L41-L53

This 180+ needs to be updated to 200+

joshdarby commented 5 years ago

@MirandaEcho I will look into this on Monday. Not sure what's going on with the staging site styles. My local looks fine on the same branch. Also, I compared the staging sites css files with my local and they're identical.

joshdarby commented 5 years ago

Looks like the reason this is being whacky on the staging environment is because of the WP Event Manager plugin. If it's active, the buttons inherit some styles from that plugin. If it's disabled, they look fine. Prod and my local both have the plugin disabled, which explains why the issue is not seen on either environment.

benlk commented 5 years ago

Does the plugin work with those styles dequeued?

MirandaEcho commented 5 years ago

Or, can we re-sync the staging site so that its back to the older version of the Events Manager Plugin and then make the changes to the numbers?

joshdarby commented 5 years ago

@benlk I'd think so. Looks like it's the bootstrap file they're loading in that's causing the issues. I'll try dequeuing it.

joshdarby commented 5 years ago

Actually, we probably don't even have to go that far. They (WP Event Manager plugin) have an option on their settings page to not enqueue Bootstrap on the frontend.

Screen Shot 2019-04-29 at 11 57 50 AM

joshdarby commented 5 years ago

Closing out this issue since the update has been deployed.