Automattic / themes

Free WordPress themes made by Automattic for WordPress.org and WordPress.com.
https://themeshaper.com
GNU General Public License v2.0
847 stars 342 forks source link

Dara: Comments Form not loading on Safari when blocks in comments is enabled. #7926

Closed devNigel closed 1 day ago

devNigel commented 2 days ago

Quick summary

The comments form fails to load on Safari browser if the blocks in comments are enabled when using the theme Dara.

The issue is reproducible on a simple site with Dara theme on Safari browser. I was not able to reproduce this issue with other themes.

Atomic sites currently don't have the option to enable blocks in comments and hence, the issue was not reproducible in Atomic.

Steps to reproduce

  1. On a simple site, activate the Dara theme.
  2. Ensure "Enable blocks in comments." is activated in Settings > Discussion.
  3. On Safari browser, go to a post or page where comments is enabled.
  4. Click on the comment form. You will notice that the form fails to load properly.

Screen recording:

On Safari: https://github.com/Automattic/themes/assets/33957570/0e4ae60f-dec6-4732-b9f0-c4250b070323

On Google Chrome: https://github.com/Automattic/themes/assets/33957570/61e04511-b3f4-4c86-813e-8a2360f05e75

What you expected to happen

Expected comments form to load properly on Safari browser.

What actually happened

Comments form failed to load properly on Safari browser.

Browser

Apple Safari

Context

No response

Platform (Simple, Atomic, or both?)

Simple

Other notes

User report: 8435655-zd-a8c

Reproducibility

Consistent

Severity

Some (< 50%)

Available workarounds?

Yes, easy to implement

Workaround details

To workaround this issue, user can disable "Enable blocks in comments." is activated in Settings > Discussion.

github-actions[bot] commented 2 days ago

Support References

This comment is automatically generated. Please do not edit it.

mrfoxtalbot commented 1 day ago

This seems to be a duplicate, closing in favor of https://github.com/Automattic/themes/issues/7931

devNigel commented 1 day ago

@mrfoxtalbot Thank you. But to be frank, this issue was opened first. The other issue should've been technically merged to this issue