-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Large banners need mobile versions #3195
Comments
Could you add a pull request with this behaviour? |
We have done like the code below using Note that we made this before translation of images is possible, right now it does not make sense to add a Norwegian image and other image (English). @ludoboludo what do you think of the picture approach? And also with regards to OP's mention of 2 images, we do not use the 2 image feature at all, I might be wrong, but think it is a much less important use case than 1 desktop + 1 mobile image. I feel like we still get something wrong though: {{ Generates and image tag with a lot of attributes. So specifically here I think we miss something:
The whole section:
|
I agree in failing to see any valid use case for 2 images in the way dawn expects. Thanks for the code snippet @gregjotau! I'll have to try it, I was building out a separate hero section due to the existing styling and hierarchical complexity of the existing image banner component, starting fresh would be easier haha. |
Yeah I agree that finding an optimal way to use the
|
Yeah, that is why it does not make sense anymore, just add a "translation image", and you are good to go :) - So that is fixed at a platform level, the code above is a hack. Without the unneeded language image code this works, but is not perfect:
|
Please, somebody enlighten me,
I would rather use a default config file with breakpoints link-able to any sections / enhanced by use cases. |
I addressed this by adding a "Visibility" setting to relevant sections, with the options: |
@ludoboludo any plan to put this functionality into Dawn? It is basically needed by everyone, as everyone has desktop and mobile customers. |
Hey @gregjotau I don't see it being added soon to Dawn as our focus is spread. But still something that I agree would be super useful to have. Ill keep relaying the information and need for it 👍 I also like the approach mentioned in this article where using the picture element, you can refer to two different image setting. |
Describe the enhancement you'd like
One feature that's a must for Shopify themes (especially in my company's usage) is for banners, slideshows, and slideshows there should be an option for mobile imagery.
This would be good for showing landscape images on desktop, and optimized square/portrait imagery for mobile viewing. It's annoying having a wide landscape image on desktop be handled so poorly on mobile.
Right now dawn offers 2 images and this is in my opinion very poorly implemented and provides little utility.
Describe alternatives you've considered
I could add this behavior myself but I'd rather this be a native feature.
Additional context/screenshots
The text was updated successfully, but these errors were encountered: