Skip to content

Added safari extension template#26

Open
donato-fiore wants to merge 7 commits into
theos:masterfrom
donato-fiore:master
Open

Added safari extension template#26
donato-fiore wants to merge 7 commits into
theos:masterfrom
donato-fiore:master

Conversation

@donato-fiore

@donato-fiore donato-fiore commented Jun 25, 2022

Copy link
Copy Markdown
Contributor

What does this implement/fix? Explain your changes.

adds a Safari extension template

Does this close any currently open issues?

no

Any relevant logs, error output, etc?
no


Any other comments?

no

Where has this been tested?

Operating System:
iOS 15.1.1
WSL - Ubuntu 20.04

Platform:
iOS/Linux

Target Platform:
iOS/iPadOS

Toolchain Version:
clang version 10.0.0 (git@github.com:apple/llvm-project.git 3093af41dd65ad466dcd5603e9289244edfee4f5)

SDK Version:
iOS 14.4

@kirb kirb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really excited to finally have some plugin templates, thanks for doing this 👍

Just had a few suggestions after looking over it again.

Comment thread ios/safari_extension/SafariWebExtensionHandler.swift Outdated
Comment thread ios/safari_extension/Makefile
Comment thread ios/safari_extension/Resources/manifest.json Outdated
Comment thread ios/safari_extension/SafariWebExtensionHandler.swift Outdated
@L1ghtmann

Copy link
Copy Markdown
Member

This looks like it fulfills Lillie's request (#25)

donato-fiore and others added 5 commits July 5, 2022 10:18
Comment thread ios/safari_extension/NIC/control.pl Outdated
NIC->variable("PARENTPROJECTNAME") = "";

while(NIC->variable("PARENTPROJECTNAME") eq "") {
NIC->variable("PARENTPROJECTNAME") = NIC->prompt("PROJ", "Name of parent project (required)");

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It may be worth specifying that by parent project name we mean an application name

@L1ghtmann

Copy link
Copy Markdown
Member

@LillieH1000 would it be possible for you to confirm the functionality of this template? I am not familiar enough with safari extensions to be able to judge myself

@LillieH1000

Copy link
Copy Markdown

@LillieH1000 would it be possible for you to confirm the functionality of this template? I am not familiar enough with safari extensions to be able to judge myself

There was one change that @kirb was supposed to make back in 2022 to fix a bundle id issue of theos generating the wrong bundle id for it but besides that it was working in 2022. I'm in the theos discord and can let you know of the exact bundle id thing to make it easier if you want?

@L1ghtmann

Copy link
Copy Markdown
Member

That would be great. Thank you!

@L1ghtmann

L1ghtmann commented Nov 23, 2024

Copy link
Copy Markdown
Member

Adjusted with the changes shared in the Theos discord this past summer. @LillieH1000 would you mind giving this a go to confirm that the issue(s) you identified are fixed? Thank you!

@LillieH1000

Copy link
Copy Markdown

Adjusted with the changes shared in the Theos discord this past summer. @LillieH1000 would you mind giving this a go to confirm that the issue(s) you identified are fixed? Thank you!

Unfortunately I no longer own an iOS device/dev setup, so I have no proper way of testing this anymore, sorry :(

@L1ghtmann L1ghtmann linked an issue Dec 8, 2024 that may be closed by this pull request
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.

[Feature Request] iOS 15 Safari Extension Template

4 participants