Skip to content
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

Merged
merged 17 commits into from
Dec 27, 2023

Conversation

jaresty
Copy link
Contributor

@jaresty jaresty commented Dec 7, 2023

  • this is heavily borrowed from the Chrome support

- this is heavily burrowed from the Chrome support
apps/arc/arc.talon Outdated Show resolved Hide resolved
apps/arc/arc.talon Outdated Show resolved Hide resolved
jaresty and others added 7 commits December 9, 2023 14:16
- 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 some reason it does not input the text into the little arc window
- this makes it unusable
- This is required in order for little arc to get focused
@jaresty jaresty requested a review from david-tejada December 12, 2023 02:45
@david-tejada
Copy link
Contributor

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.

Copy link
Collaborator

@fidgetingbits fidgetingbits left a 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.

Copy link
Collaborator

@auscompgeek auscompgeek left a 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
@jaresty
Copy link
Contributor Author

jaresty commented Dec 16, 2023

Sure thing. I don't have a windows box to test so I'll rename them.

- properly restrict talonscript files to mac
@jaresty jaresty requested a review from auscompgeek December 18, 2023 00:59
@fidgetingbits
Copy link
Collaborator

I think this one is ok to merge now. The additional changes to mac only lgtm.

@fidgetingbits fidgetingbits merged commit bbcd1b8 into talonhub:main Dec 27, 2023
1 check passed
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.

4 participants