Skip to content

Update safari-create-reading-list-item.applescript#1030

Merged
grzegorzkrukowski merged 3 commits intoraycast:masterfrom
JNBARY:patch-1
Apr 15, 2026
Merged

Update safari-create-reading-list-item.applescript#1030
grzegorzkrukowski merged 3 commits intoraycast:masterfrom
JNBARY:patch-1

Conversation

@JNBARY
Copy link
Copy Markdown
Contributor

@JNBARY JNBARY commented Mar 31, 2026

Description

Modified the script to optionally use the URL of the webpage currently open in Safari as Link for the new Reading List item.

Type of change

  • Improvement of an existing script

Screenshot

By using "open" for the "Link"-argument the webpage currently open in Safari will be added to the Reading List.

Dependencies / Requirements

No additional steps needed during installation.

Checklist

Modified the script to optionally use the URL of the webpage currently open in Safari as Link for the new Reading List item.
Copy link
Copy Markdown
Contributor

@unnamedd unnamedd left a comment

Choose a reason for hiding this comment

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

Hi @JNBARY, thanks for the contribution to the repository.

There's only one thing that, in my opinion, you need to improve: the placeholder message that will be shown in the root search. Even though the Script Command here is a Safari one, it is important to add the word "Safari" to avoid confusion. Additionally, you should clarify which word will be used in your action.

However, on second thought, I believe it would be much better if, instead of requiring the user to type "open" (which is essentially giving them an extra task) to add the page to the Reading List, you might want to discard this PR and create a separate Script Command exclusively for that purpose. The user would type into the root search: "Add Open Page to Reading List" and voilà, everything is set.

What do you think?

Comment thread commands/apps/safari/safari-create-reading-list-item.applescript Outdated
Co-authored-by: Thiago Holanda <unnamedd@gmail.com>
Copy link
Copy Markdown
Contributor

@unnamedd unnamedd left a comment

Choose a reason for hiding this comment

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

Only one last thing here I forgot to ask you to add.

Comment thread commands/apps/safari/safari-create-reading-list-item.applescript
Co-authored-by: Thiago Holanda <unnamedd@gmail.com>
Copy link
Copy Markdown
Contributor

@unnamedd unnamedd left a comment

Choose a reason for hiding this comment

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

Nice! It is approved!

As soon as our internal friends at Raycast can, they will merge it! :D

@grzegorzkrukowski grzegorzkrukowski merged commit eb4b455 into raycast:master Apr 15, 2026
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.

3 participants