Skip to content

Fixed a bug in Scroll.py#2524

Open
djp3 wants to merge 1 commit intoZulko:masterfrom
djp3:patch-1
Open

Fixed a bug in Scroll.py#2524
djp3 wants to merge 1 commit intoZulko:masterfrom
djp3:patch-1

Conversation

@djp3
Copy link
Copy Markdown

@djp3 djp3 commented Sep 30, 2025

Fix a bug in the Scroll Effect.

Basically it wasn't working. The scroll effect was stopping before it should and would sometimes freak out and just start rolling over the content really quickly.

  • [✅] I have provided code that clearly demonstrates the bug and that only works correctly when applying this fix
  • [✅ ] I have added suitable tests demonstrating a fixed bug or new/changed feature to the test suite in tests/
  • [✅] I have properly documented new or changed features in the documentation or in the docstrings
  • [✅ ] I have properly explained unusual or unexpected code in the comments around it

Here is a python program that contains the corrected code that can be run against the current code to show the bugs.
test_scroll_effect.py

I put a stripped down version of that file into the tests directory.

@djp3 djp3 marked this pull request as ready for review September 30, 2025 19:37
@keikoro
Copy link
Copy Markdown
Collaborator

keikoro commented Oct 22, 2025

@djp3 Please include any "fixes" to your own PR within the original commit(s), i.e. rebase instead of tacking on more commits. And use descriptive commit messages for the original changes.

… over was too large or too small for the given amount of time.
@djp3
Copy link
Copy Markdown
Author

djp3 commented Nov 19, 2025

Tried to squash it as you requested. I'm not 100% sure that I got the workflow right though.

@djp3
Copy link
Copy Markdown
Author

djp3 commented Jan 14, 2026

Clean merge for the win!

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.

2 participants