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

Investigate and improve on-update behavior in Firefox (again) #1550

Closed
marcustyphoon opened this issue Jul 18, 2024 · 2 comments
Closed

Investigate and improve on-update behavior in Firefox (again) #1550

marcustyphoon opened this issue Jul 18, 2024 · 2 comments
Labels
enhancement New feature or request stale Marked for closure due to inactivity

Comments

@marcustyphoon
Copy link
Collaborator

Concept

With the release of v1.0.1 to Firefox, I'm seeing various reports that the features silently stop working until you refresh/open a new tab. Reports include the radar hiding tweak, hide my own posts on the dash tweak, possibly No Recommended sidebar hide...

Unfortunately, to my knowledge, all of these work fine when you simply toggle the extension off and on in Firefox; hopefully this isn't impossible to reproduce in development. If it is possible to reproduce in development, we can probably(?) fix it. For example, maybe the script element used to initialize the main world handlers needs to not have type: module or something, since module scripts don't get executed for side effects multiple times. Ah, damn, that's probably it, isn't it.

(Wait... I guess one can just reproduce this by installling the 1.0.0 xpi, opening Tumblr, then firing the update in a new tab. Nice. How one can test a fix, I'm not so sure.)

@marcustyphoon marcustyphoon added the enhancement New feature or request label Jul 18, 2024
@marcustyphoon marcustyphoon changed the title Investigate on-update behavior in Firefox (again) Investigate and improve on-update behavior in Firefox (again) Jul 18, 2024
@marcustyphoon
Copy link
Collaborator Author

Aside: Oh, hey, toggling the extension duplicates the Quote Replies action button. Only the new one works, since we attach the listener directly to the element, so it's not like a check that a button already exists would help in this case; we need to remove it.

Perhaps we should add element.dataset.xkitRewritten = ''; to the dom util and just get rid of everything, not just buildStyle style elements, on boot. Clutters the dev tools, though.

Copy link

stale bot commented Aug 18, 2024

This issue is unconfirmed, and has been labelled as stale due to inactivity. It will be closed automatically if no further activity occurs.

A project maintainer can mark an issue as confirmed by adding the help wanted label, the wontfix label, or an assignee.

@stale stale bot added the stale Marked for closure due to inactivity label Aug 18, 2024
@stale stale bot closed this as completed Aug 26, 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 stale Marked for closure due to inactivity
Projects
None yet
Development

No branches or pull requests

1 participant