adds .webloc support - Githubissues

gingerbeardman / stapler

My take on the classic Macintosh app Stapler (with a little bit of LaunchList)

https://blog.gingerbeardman.com/2024/08/10/stapler-i-remade-a-32-year-old-classic-macintosh-app/

MIT License

238 stars 10 forks source link

adds .webloc support #14

Open kristofer opened 3 months ago

kristofer commented 3 months ago

Adds .webloc support for dropping a web url from browser into a .stapled file. Launching works as well without mods.

It allows a user to include web bookmarks into Stapler files, along with the local files.

kristofer commented 3 months ago

I guess I should add, I was unable to get the weblog stuff to work on 13.6 Ventura. But I see your example in the readme has a weblog item in it.

gingerbeardman commented 3 months ago

Thanks for this!

I'd love it if you could remove unrelated files from this commit, to make it easier for me to merge.

.pbxproj
IDEWorkspaceChecks.plist
.xcbkptlist
.xcuserstate
etc.

The goal would be to keep it as minimal as possible! Thanks @kristofer

gingerbeardman commented 3 months ago

Also, sorry this will need to be rebased as I've just implemented some changes to how things are added and launched—sorry!

gingerbeardman commented 3 months ago

I guess I should add, I was unable to get the weblog stuff to work on 13.6 Ventura. But I see your example in the readme has a weblog item in it.

My webloc item was dragged from Safari > Finder > Stapler

kristofer commented 3 months ago

finally seeing this, will do. just give me a day or few. thanks! (ps and sorry, i thought i had reduced it to just the one file)

gingerbeardman commented 3 months ago

No rush at all! I think this change will also get us the ability to drag and drop email addresses in the same way #36

kristofer commented 3 months ago

Glad you asked me to clean this up.

So the .webloc I'm saving in the StaplerDoc is merely a file (containing the actual target URL) in the temporary directory of the app. SO, when that gets deleted, the weblog file is gone. Bad.

I think there are two options.

make AliasItem handle other things than just file-urls - like web urls and email addresses and what-not. the alias would have a content-type field, for each species (MIME types?)
make a place where .webloc (and other non-file-urls) get copied when created so they aren't lost when the TMP dir changes. (yuck!)

hmm.

kristofer commented 3 months ago

Since the alias just gets JSON'd when saved, seems like adding typing to them makes the most sense.

gingerbeardman commented 3 months ago

See https://github.com/gingerbeardman/stapler/issues/18 for an explanation of filesystem bookmarks and an example of the data they store.

I wonder if we can use webloc:// (or either http:// or https://) and mailto:// URLs?

I'll give it some thought. I might even test a hand-crafted bookmark.

kristofer commented 3 months ago

lemme untangle this mess

kristofer commented 3 months ago

okay, I think I've cleaned it all up, along with pulling in all your latest commits. @gingerbeardman

gingerbeardman commented 3 months ago

Will take a look ASAP!