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

Holiday snow causes jitter/shake on sites with ar/Arabic language #40569

Open
philnick206 opened this issue Dec 11, 2024 · 4 comments
Open

Holiday snow causes jitter/shake on sites with ar/Arabic language #40569

philnick206 opened this issue Dec 11, 2024 · 4 comments
Assignees
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Experiment] AI labels added [mu wpcom Feature] Holiday Snow [Package] Jetpack mu wpcom WordPress.com Features [Plugin] mu wpcom jetpack-mu-wpcom plugin [Pri] Normal Triaged [Type] Bug When a feature is broken and / or not performing as intended

Comments

@philnick206
Copy link

philnick206 commented Dec 11, 2024

Impacted plugin

Jetpack

Quick summary

Activating holiday snow on a site with "ar" set as the site language will cause a jitter/shake to appear when viewing the site in a browser.

Image

Steps to reproduce

  1. Activate holiday snow on site
  2. Ensure language is set to 'ar'
  3. View site in a browser

A clear and concise description of what you expected to happen.

Snow should be visible but nothing else

What actually happened

Snow is visible but horizontal bars on browser flicker (even though there is no content off the page), screen can appear to shake

CleanShot.2024-12-10.at.19.31.04.mp4

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

If the above answer is "Yes...", outline the workaround.

No response

Platform (Simple and/or Atomic)

No response

Logs or notes

Tested on Twenty Twenty Four theme.
Reported in 9159191-zd-a8c

@philnick206 philnick206 added [mu wpcom Feature] Holiday Snow [Package] Jetpack mu wpcom WordPress.com Features [Plugin] mu wpcom jetpack-mu-wpcom plugin [Type] Bug When a feature is broken and / or not performing as intended Needs triage Ticket needs to be triaged labels Dec 11, 2024
@philnick206 philnick206 changed the title Holiday snow causes jitter/shake on sites with Arabic language Holiday snow causes jitter/shake on sites with ar/Arabic language Dec 11, 2024
@github-actions github-actions bot added [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] High [Feature Group] Support All things related to customer support. [Feature] Jetpack labels Dec 11, 2024
Copy link
Contributor

OpenAI suggested the following labels for this issue:

  • [Feature Group] Support: The issue relates to a bug that could affect user experience, which falls under support-related concerns.
  • [mu wpcom Feature] Holiday Snow: The problem directly involves the holiday snow feature, which is the main subject of the issue.
  • [Feature] Jetpack: Since the affected plugin is Jetpack, this label indicates that the problem is related to its functionalities.

Copy link
Contributor

Support References

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

  • 9159191-zen

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". label Dec 11, 2024
@Robertght
Copy link

Robertght commented Dec 11, 2024

📌 REPRODUCTION RESULTS

  • Tested on Simple – Replicated
  • Tested on Atomic – Replicated

📌 FINDINGS/SCREENSHOTS/VIDEO

  • Replicated, but I don't see a screen shake, but a horizontal scroll that appears briefly from time to time.

📌 ACTIONS

  • Triaged
  • I lowered the priority to normal.

@jeherve I'm cc'ing you as I don't know who is the snow owner 😆

@Robertght Robertght added Triaged and removed Needs triage Ticket needs to be triaged labels Dec 11, 2024
@Robertght Robertght moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Dec 11, 2024
@jeherve jeherve removed [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Feature Group] Support All things related to customer support. [Status] Escalated to Product Ambassadors [Feature] Jetpack labels Dec 11, 2024
@jeherve jeherve self-assigned this Dec 11, 2024
@jeherve
Copy link
Member

jeherve commented Dec 13, 2024

I can reproduce, it happens with any RTL language. It would appear that the document width is bumped with the flakes, thus causing the horizontal scrollbar to come up.

This is something that would need to be fixed in the upstream library I'm afraid. I attempted to play with some of the customization options (namely flakeLeftOffset and flakeRightOffset, but no luck. I may try to play with this some more in the future, but realistically it may make more sense to switch to a different implementation entirely, possibly a CSS-only one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Experiment] AI labels added [mu wpcom Feature] Holiday Snow [Package] Jetpack mu wpcom WordPress.com Features [Plugin] mu wpcom jetpack-mu-wpcom plugin [Pri] Normal Triaged [Type] Bug When a feature is broken and / or not performing as intended
Projects
Development

No branches or pull requests

3 participants