Update safari-create-reading-list-item.applescript#1030
Update safari-create-reading-list-item.applescript#1030grzegorzkrukowski merged 3 commits intoraycast:masterfrom
Conversation
Modified the script to optionally use the URL of the webpage currently open in Safari as Link for the new Reading List item.
unnamedd
left a comment
There was a problem hiding this comment.
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?
Co-authored-by: Thiago Holanda <unnamedd@gmail.com>
unnamedd
left a comment
There was a problem hiding this comment.
Only one last thing here I forgot to ask you to add.
Co-authored-by: Thiago Holanda <unnamedd@gmail.com>
unnamedd
left a comment
There was a problem hiding this comment.
Nice! It is approved!
As soon as our internal friends at Raycast can, they will merge it! :D
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
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