-
Notifications
You must be signed in to change notification settings - Fork 800
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
Adds basic support for the Arc web browser #1326
Adds basic support for the Arc web browser #1326
Conversation
jaresty
commented
Dec 7, 2023
•
edited
Loading
edited
- this is heavily borrowed from the Chrome support
- this is heavily burrowed from the Chrome support
- If you say please followed by some text it will search for that in the command palette - If you just say please it will only open the command palette
for more information, see https://pre-commit.ci
- for some reason it does not input the text into the little arc window - this makes it unusable
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
I added a comment. By the way, I don't have write access so my review won't be enough for the "review required" criteria. But don't worry, someone will review it and merge it when they have time. |
- this should ensure that arc has time to respond
for more information, see https://pre-commit.ci
- adds a comment to clarify that requirement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I haven't tested, but it looks simple enough.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The keys in the Talonscript appear to be Mac-specific. Given Arc is now available for Windows, could you please either implement the commands for Windows too, or move the commands to Mac-specific files?
- these are mac specific
Sure thing. I don't have a windows box to test so I'll rename them. |
- properly restrict talonscript files to mac
I think this one is ok to merge now. The additional changes to mac only lgtm. |