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

[DOES NOT WORK] Vain attempts at getting noscript streaming to work #77

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheTechRobo
Copy link
Owner

@TheTechRobo TheTechRobo commented Feb 16, 2024

This PR contains my broken and not-quite-finished support for noscript streaming.

It's so fucking lame what I got stuck on: the link to the archived version cannot be added using the CSS content property.

I'm keeping this PR open with the glimmer of hope that maybe I or someone else will think of something - or maybe position: running will finally have proper browser support (I think it would be a solution to this problem, but it looks like it may only be available to page margin boxes).

Would partially f​i​x #66

@TheTechRobo TheTechRobo marked this pull request as draft February 16, 2024 02:27
@TheTechRobo
Copy link
Owner Author

There are two options I can think of from here, unless something radically amazing comes up (a way to change an <a> or add one where it should be):

  1. Send the loading messages in a div with a unique ID, then set display:none and generate another once there is new information (VERY inefficient client-side as despite not being rendered the previous divs would still be part of the DOM)
  2. Make a long-lived request for an iframe for each of the service responses that will eventually resolve to an <a> (Inefficient server-side with memory usage as the requests will last so long; leads to potential memory issues, performance issues (so many coroutines), and browser timeouts)

Both of the available options seem subpar to me.

@TheTechRobo TheTechRobo added enhancement New feature or request module:frontend help wanted Extra attention is needed stumped Difficult problem labels Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed module:frontend stumped Difficult problem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant