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

Allow disabling the hidden preferences window. #2269

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stuckj
Copy link

@stuckj stuckj commented Dec 22, 2024

This gives an option under advanced settings allowing the user to disable the hidden preferences window when applying settings (i.e., adding the hide-preferences-window class when caprine opens settings). This will be disabled by default (current behavior). If enabled, you'll very briefly see the settings window flash when caprine applies a setting inside Messenger.

Based on conversation here: #2265

This gives an option under advanced settings allowing the user to
disable the hidden preferences window when applying settings (i.e.,
adding the hide-preferences-window class when caprine opens settings).
This will be disabled by default (current behavior). If enabled, you'll
very briefly see the settings window flash when caprine applies a
setting inside Messenger.
@mquevill
Copy link
Collaborator

I think this is a good way to implement a workaround, rather than having users remove the hide-preferences-window class from the dev tools area like is mentioned in multiple other issues. When Facebook (inevitably) changes selectors again, this workaround would definitely be more user-friendly in the meantime.

@stuckj
Copy link
Author

stuckj commented Jan 1, 2025

Cool. I'm guessing the process is to merge is someone with merge permissions will eventually review and merge this if accepted or come back with comments if there are changes requested? I looked through the contributing guidelines below and believe this should be good for review and merge whenever someone has a chance to take a look. :)

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