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

Footer has extra empty space #2691

Closed
rozinashopify opened this issue Jun 2, 2023 · 3 comments · Fixed by #2745
Closed

Footer has extra empty space #2691

rozinashopify opened this issue Jun 2, 2023 · 3 comments · Fixed by #2745

Comments

@rozinashopify
Copy link

rozinashopify commented Jun 2, 2023

Describe the current behavior

When unchecking the "Show email signup" setting in the footer the space and lines around it stays in the footer.

Describe the expected behavior

This extra space and lines should not be there when the email sign up is turned off.

Version information (Dawn, browsers and operating systems)

  • Dawn Version: 9.0.0
  • Chrome Version 114.0.5735.90
  • macOS Version 13.4

Possible solution

Additional context/screenshots

https://screenshot.click/31-18-8cc0e-9qmzb.png

@rozinashopify rozinashopify added the Category: Bug Something isn't working label Jun 2, 2023
@qui-gon-jon
Copy link
Contributor

Hi @rozinashopify

It looks like the space is only appearing because both the "Enable Follow on Shop" and "Show social media icons" settings are checked, but neither are configured. The space is reserved for social media icons and the follow on shop button. If you disable both of those settings as well, then the extra space is removed: Screen recording

@rozinashopify
Copy link
Author

I don't have any social media icons set in the theme, in this case the container shouldn't show up at all.

@qui-gon-jon
Copy link
Contributor

I see your point and I agree. Right now if your social media icons are not set, you would also need to disable this checkbox in order to remove the white space.

But it would be more intuitive if we check for the "enable" setting, and also that there are social icons configured. I think we can solve this by editing this condition in sections/footer.liquid to add and has_social_icons == true

  {%- if section.blocks.size > 0
    or section.settings.newsletter_enable
    or section.settings.show_social and has_social_icons == true
    or section.settings.enable_follow_on_shop
  -%}

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

Successfully merging a pull request may close this issue.

4 participants