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

MAINT: rename icons section to avoid adblock filtering #218

Merged
merged 1 commit into from
Mar 29, 2023

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented Mar 2, 2023

I couldn't fully test this locally as I was getting a make error, but I am almost certain this would fix the issue I see, namely that I don't see the community and social media icons in the footer.

@netlify
Copy link

netlify bot commented Mar 2, 2023

Deploy Preview for scientific-python-hugo-theme ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit b7f4ae6
🔍 Latest deploy log https://app.netlify.com/sites/scientific-python-hugo-theme/deploys/6400475c901da700086de794
😎 Deploy Preview https://deploy-preview-218--scientific-python-hugo-theme.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@bsipocz
Copy link
Member Author

bsipocz commented Mar 2, 2023

Indeed, I see them in the rendering above, while I don't see them in the current deployment at https://theme.scientific-python.org/ . 🎉

Screenshot 2023-03-01 at 22 54 57

@bsipocz
Copy link
Member Author

bsipocz commented Mar 29, 2023

I go ahead and merge, so I'll have the fix :)

@bsipocz bsipocz merged commit 52c3362 into scientific-python:main Mar 29, 2023
@jarrodmillman jarrodmillman added this to the 0.1 milestone Sep 29, 2023
@bsipocz bsipocz deleted the rename_social_media branch September 26, 2024 22:45
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.

3 participants