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

Allow setting GIFS as author image #394

Merged
merged 2 commits into from
Aug 10, 2021
Merged

Conversation

Rihoj
Copy link
Contributor

@Rihoj Rihoj commented Aug 10, 2021

Issue

fixes #392
fixes #209

Description

allow_gifs - don't apply image processing to svg or gif

Test Evidence

Add a gif as a author image, and make sure it displays as expected on home page as well as single posts or anywhere else the author image is used.

Copy link
Member

@hossainemruz hossainemruz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Rihoj

@hossainemruz hossainemruz changed the title allow_gifs Allow setting GIFS as author image Aug 10, 2021
@kodiakhq kodiakhq bot merged commit fc3b483 into hugo-toha:main Aug 10, 2021
@Rihoj
Copy link
Contributor Author

Rihoj commented Aug 10, 2021

Glad to be of help @hossainemruz !

@Rihoj Rihoj deleted the allow_gifs branch August 10, 2021 02:42
@talves
Copy link

talves commented Jun 14, 2022

I believe this PR broke your theme. The assignment syntax "=" is incorrect from what I can tell. I don't have time to make an issue, because I'm not actually using the theme, but wanted to give you a heads up.

@Rihoj
Copy link
Contributor Author

Rihoj commented Jun 14, 2022

@talves I will take a look when I can. Thank you for letting me know

@talves
Copy link

talves commented Jun 14, 2022

Remember when short declaration operator and trying to change it’s value type you would typically get an error. I think there's something going on with that here in the code. It may have always been there.

https://golangdocs.com/short-variable-declaration-in-golang

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