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

Add Browser Utilities Extension #614

Closed
wants to merge 5 commits into from
Closed

Add Browser Utilities Extension #614

wants to merge 5 commits into from

Conversation

NexusKitten
Copy link
Contributor

@NexusKitten NexusKitten commented Jun 19, 2023

A handful of blocks for receiving and setting browser information.

image
image

It should go without saying that most if not all of these blocks will not function when the project is packaged into an exe file.

@LilyMakesThings
Copy link
Contributor

4 of these blocks exist within other extensions; in network (dark green) they're far more precise. Consider revising the existing blocks to at least match the functionality.

image

@LilyMakesThings
Copy link
Contributor

It's nice to finally see alert, confirm and prompt have a home outside of LMS Utils :') only a few more blocks until that extension is finally obsolete! woohoo!

@GarboMuffin
Copy link
Member

I think I will make isonline just check navigator.online. It's not great, but it doesn't involve making requests to a website that many schools block for one reason or another

@GarboMuffin
Copy link
Member

this would probably be not very affected by #674

@NexusKitten NexusKitten changed the title Add Browser Utilities Extension Add Bowser's Utilities Extension Jul 15, 2023
@NexusKitten NexusKitten changed the title Add Bowser's Utilities Extension Add Browser Utilities Extension Jul 17, 2023
@GarboMuffin
Copy link
Member

i wonder if we just rename navigator to browser utilities and put everything from this in there

@NexusKitten

This comment was marked as outdated.

@NexusKitten

This comment was marked as spam.

@NexusKitten NexusKitten reopened this Aug 21, 2023
@NexusKitten

This comment was marked as spam.

@NexusKitten

This comment was marked as spam.

It was rather outdated
@NexusKitten

This comment was marked as spam.

@LilyMakesThings
Copy link
Contributor

@GarboMuffin any reason lint isn't running for me?

You have conflicts

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.

4 participants