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

Bug - featured image - featured image is getting hidden randomly #487

Open
SophieAmnesty opened this issue Nov 21, 2024 · 7 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@SophieAmnesty
Copy link

Describe the bug
I have noticed this a few times recently but I have no idea what's triggering it.

Sometimes the featured image on a post will get hidden and nobody seems to have meant that to happen.

I will go into the post and find the hide featured image setting toggled on, without me or the post editor having selected that setting.

It could just be user error, but it's sort of a difficult setting to change by accident.

It happened on these posts:
https://www.amnesty.org/en/latest/news/2024/11/ethiopia-end-the-month-long-arbitrary-detention-of-thousands-in-amhara-region/
https://www.amnesty.org/en/latest/news/2023/10/damning-evidence-of-war-crimes-as-israeli-attacks-wipe-out-entire-families-in-gaza/

To Reproduce
I haven't figured out how to reproduce this yet. Hopefully I'll be able to come back and add detail to this at some point.

Expected behaviour
Featured image shouldn't be hidden unless the hide featured image setting has been selected by a user.

Actual behaviour
Sometimes the featured image seems to be getting hidden randomly.

@SophieAmnesty
Copy link
Author

Sorry this is a bit impossible to diagnose but I have been noticing it happen a bit recently. Will add any more info to this ticket if/when I can!

@SophieAmnesty
Copy link
Author

@jaymcp
Copy link
Contributor

jaymcp commented Nov 27, 2024

sorry for the slow response on this, @SophieAmnesty.

the only thing in the code that can toggle that option without user interaction is the hero block - if a hero is inserted into a post, display of the featured image is disabled, so that the post doesn't render both the featured image and the hero. but then, if the hero is subsequently removed, we don't toggle the option back.

This is an oversight, but perhaps we should avoid meddling with the display of the featured image at all in the hero block, and leave the decision up to the user. what do you think?

@SophieAmnesty
Copy link
Author

Ahh yes ok I see.

Yeah maybe we should do that and then see if the mysterious hidden featured image thing stops happening?

then we can assume people were just adding and removing a hero block.

@jaymcp jaymcp moved this from To do to PR Created (ready for Review) in Humanity Web Development Nov 29, 2024
@jaymcp jaymcp assigned jaymcp and unassigned AmnestyAM Nov 29, 2024
@SophieAmnesty
Copy link
Author

SophieAmnesty commented Dec 3, 2024

happened again with this post: https://www.amnesty.org/en/latest/campaigns/2024/12/if-society-is-to-change-people-with-disabilities-must-be-included-in-the-fight/

I asked the user if she'd added the hero block and removed it and she said she's never used a hero block in her life.

So, I'm not 100% sure this fix of removing the hook from the hero block to hide featured image is going to work.

we shall see I guess!

@jaymcp
Copy link
Contributor

jaymcp commented Dec 3, 2024

Oh, that's unexpected. This is quite the perplexing issue! Thanks for the new info 🙂

@Gentyspun Gentyspun moved this from PR Created (ready for Review) to PR Approved (ready for DEV) in Humanity Web Development Dec 5, 2024
@SophieAmnesty
Copy link
Author

@jaymcp jaymcp moved this from PR Approved (ready for DEV) to Merged to DEV (ready for Internal Testing) in Humanity Web Development Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Merged to DEV (ready for Internal Testing)
Development

No branches or pull requests

3 participants