Skip to content

feat: adds monitor_from_point method#9770

Merged
amrbashir merged 14 commits intotauri-apps:devfrom
clearlysid:feat/monitor-from-point
May 14, 2024
Merged

feat: adds monitor_from_point method#9770
amrbashir merged 14 commits intotauri-apps:devfrom
clearlysid:feat/monitor-from-point

Conversation

@clearlysid
Copy link
Contributor

@clearlysid clearlysid commented May 14, 2024

Exposes this Tao method in Tauri too. Merging this closes #3057 and unblocks spyglass-search/spyglass#312

@clearlysid
Copy link
Contributor Author

If someone can guide me to resolve the above comments, this PR will be ready to review 💫

@clearlysid clearlysid marked this pull request as ready for review May 14, 2024 03:07
@clearlysid clearlysid requested a review from a team as a code owner May 14, 2024 03:07
Copy link
Member

@amrbashir amrbashir left a comment

Choose a reason for hiding this comment

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

Thanks for contributing, LGTM, just need to get the documentation consistent and should be good to go. You also need to run yarn build and yarn format inside tooling/api

@clearlysid clearlysid requested a review from amrbashir May 14, 2024 16:30
@clearlysid
Copy link
Contributor Author

All good from my end ✅

amrbashir
amrbashir previously approved these changes May 14, 2024
Copy link
Member

@amrbashir amrbashir left a comment

Choose a reason for hiding this comment

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

Thank you

@amrbashir amrbashir force-pushed the feat/monitor-from-point branch from dd5c909 to 6222f0e Compare May 14, 2024 16:42
@clearlysid
Copy link
Contributor Author

clearlysid commented May 14, 2024

wow how'd you do this? here I thought I needed to go back and re-commit each one again with signature 😂 — nice work!

@amrbashir
Copy link
Member

@amrbashir amrbashir merged commit ec0e092 into tauri-apps:dev May 14, 2024
@clearlysid clearlysid deleted the feat/monitor-from-point branch May 15, 2024 04:18
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.

[feat] Monitor::from_point and Monitor::cursor_position

2 participants