Job a11n
(@jobthomas)
Automattic Happiness Engineer
Hey @rhondasellers – is it possible that you have Jetpack enabled? If so, under Jetpack > Settings, disable the infinite scroll:
Link to image: https://cld.wthms.co/XPCObl
That will likely do the trick for you.
You’re amazing!!! Thank you. I never would have guessed that. That also fixed my “sort by” menu going away. I didn’t know there were conflicts with jetpack with this theme, but that’s okay, I don’t use jetpack for much.. I did like the infinite scroll but I like the footer bar more.
So I take it i can’t move this sidebar to the top on mobile then right?? If not then I’ll mark this as resolved cuz that’s fixed most of my issues.
Job a11n
(@jobthomas)
Automattic Happiness Engineer
Hey @rhondasellers – you’re welcome.
Which sidebar do you mean? Because on mobile you don’t have a lot of space, most people prefer to have the products/content listed first. It is possible to change that with custom templating though, so if you’d feel okay with changing some PHP templates, we can get you started.
I am comfortable making PHP changes just fine. The sidebar on the shop with categories product tags and the currency converter. I could probably settle for just the currency converter being up top.
The issue is there’s so much scrolling to get to the bottom since there’s a ton of products and usually people that come to my site know at least the color what they’re looking for, sometimes material (category) *different blanks can be turned to be chunkier than others, so some just go after those ones.*
Sooo if you can find me a way to get the side bar up there i’d greatly appreciate you. I wouldn’t mind it being at the bottom if it wasn’t for the amount of inventory I have.
Thank you much!!
The sidebar displays after the main content in mobile because the main content comes first in the HTML. In order to get the sidebar to show first, the order would need to be reversed in the code. The below should do it for you:
remove_action( 'storefront_sidebar', 'storefront_get_sidebar', 10 );
add_action( 'storefront_content_top', 'storefront_get_sidebar', 20 );
This will make the sidebar appear in the code first, which should give you the desired result.
that almost works, but not quite. It works great on woocommerce pages.. but on my homepage and a lot of other pages, i have them set to be full width without the sidebar (anything that isn’t a shop page, basically, is full width)
When i add this function, I end up with the sidebar on the other pages and in a bad way on desktop. It moves all my content under a sidebar that should not be showing on that page anyway. That’s why i was hoping I could move it in CSS based on screen size but I don’t see a way to do it.
@rhondasellers Yes, I can see how that can be a problem with full width pages. Unfortunately, there isn’t a way to rearrange the content how you’re wanting with just CSS due to how the content appears in the HTML itself.
The functions here are pluggable, meaning they can be overridden in a child theme or plugin:
https://github.com/woocommerce/storefront/blob/master/inc/woocommerce/storefront-woocommerce-template-functions.php#L8-L39
If you were to override these two in your child theme and move the do_action from line 38 to be before line 18, it should move the sidebar just for WooCommerce pages, leaving all other pages the same.
Jesse, I am having another issue? My host emailed me stating that I had malicious content uploaded to the site. I checked stream to see what happened, I see that someone logged in as me and uploaded it.
When i checked the whois on the IP it links to Automattic.
Can you look into this???
@rhondasellers Did you ever provide a login to anyone is support in any way, maybe a ticket or a chat? These are the only places we would ask for login details, and without login details we have no access to your site.
We haven’t heard back from you in a while, so I’m going to mark this as resolved – if you have any further questions, you can start a new thread.
