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

Fix: BlueSky expandos have issues on chromium browsers #5561

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

steckums
Copy link
Contributor

Now that most subreddits banned twitter posts, I have been seeing more bsky.app posts pop up and have made a few changes as I have been testing them, including improving handling of url recognition.

I have also added a little error message if the embed api returns an error, as that post does not support being embedded due to user settings on the site. Here is an example:

image

I also noticed that the embedded javascript only works in Firefox. I am now stripping the script out and there is currently no way to exclude the script tag via the API like the twitter modules does. I also made the styling match the twitter expando when you don't have a tampermonkey/greesemonkey script to include the embed.js manually.

image

Unfortunately, I cannot get the embedded javascript to work unless I have the mentioned greesemonkey script. Here's what I was using: https://greasyfork.org/en/scripts/525193-res-bluesky-expandos

Relevant issue: N/A
Tested in browser: Firefox, Brave/Chrome

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.

1 participant