Skip to content

Conversation

@thesomewhatyou
Copy link
Contributor

Description

Added slow scroll.

This pull request introduces slow scroll, a funbox feature, which animates the word list and caret during a test session to create a slow upward scrolling effect, so YOU can suffer. Additionally, the funbox lifecycle is improved to ensure that any active funbox can clean up its state when a test ends or restarts, as it'd be helpful both IN THE FUTURE and now.

Checks

None of the checks applies to funbox so I'll go burn in hell.

Closes #

No

(I made a Homer theme, but had a funbox idea, and thought this would be bettah)

@monkeytypegeorge monkeytypegeorge added frontend User interface or web stuff assets Languages, themes, layouts, etc. packages Changes in local packages labels Nov 22, 2025
@thesomewhatyou thesomewhatyou marked this pull request as ready for review November 22, 2025 06:21
@github-actions github-actions bot added the waiting for review Pull requests that require a review before continuing label Nov 22, 2025
@thesomewhatyou
Copy link
Contributor Author

This is SLIGHTLY broken, but... who gives a rat's ass? New funbox mode.

@fehmer
Copy link
Member

fehmer commented Nov 22, 2025

  • does not work with tape mode
  • text scrolls out of view even if you manage to keep up with the scrolling

I think the main problem is that the scrolling speed is disconnected to the typing speed

@thesomewhatyou
Copy link
Contributor Author

thesomewhatyou commented Nov 22, 2025

  • does not work with tape mode
  • text scrolls out of view even if you manage to keep up with the scrolling

I think the main problem is that the scrolling speed is disconnected to the typing speed

Yeah... I'm pretty sure THAT was the broken part. I coded this at 1 AM, excuse me.

I'm not sure HOW to implement tape mode, so if you have any tips, would appreciate.

@thesomewhatyou thesomewhatyou marked this pull request as draft November 22, 2025 15:22
@github-actions github-actions bot removed the waiting for review Pull requests that require a review before continuing label Nov 22, 2025
@fehmer
Copy link
Member

fehmer commented Nov 22, 2025

I'm not sure HOW to implement tape mode, so if you have any tips, would appreciate.

ig tape mode could scroll horizontally. I don't know how to implement it.

…obably making it slower for our own good, and it should be fine
@thesomewhatyou thesomewhatyou marked this pull request as ready for review November 22, 2025 19:27
@github-actions github-actions bot added the waiting for review Pull requests that require a review before continuing label Nov 22, 2025
@thesomewhatyou
Copy link
Contributor Author

99% sure this works

@thesomewhatyou
Copy link
Contributor Author

i don't know how to solve merge conflicts

tenor

@thesomewhatyou
Copy link
Contributor Author

update: i now know how to resolve conflics

@thesomewhatyou
Copy link
Contributor Author

could someone run pnpm test im paranoid and i don't have any more codespace usage

@Miodec
Copy link
Member

Miodec commented Nov 26, 2025

Ill be honest im not sure i see the appeal of this. And it has caret positioning and word clipping issues. Sorry.

@Miodec Miodec closed this Nov 26, 2025
@thesomewhatyou
Copy link
Contributor Author

Ill be honest im not sure i see the appeal of this. And it has caret positioning and word clipping issues. Sorry.

yeah, i wasn't sure how to implement it

doing this felt like a fever dream since i only had time to do this either at 2 am or 4 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

assets Languages, themes, layouts, etc. frontend User interface or web stuff packages Changes in local packages waiting for review Pull requests that require a review before continuing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants