feat(controls): add tray menu and shortcut for recording

[onyedikachi-david]

description

• Add stop/start recording buttons to tray menu
• Implement global shortcut for recording control
• Add unified state management for recording status
• Improve process detection reliability
• Add proper error handling and user feedback
• Also fixed the issues in the previous pull request Feat/tray menu/shortcut recording #552

related issue: #713
/claim #713
Fixes: #713

Screen.Recording.2024-12-05.at.02.44.32.mov

Screen.Recording.2024-12-05.at.02.49.19.mov

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
screenpipe	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 6, 2024 8:12pm

[louis030195]

i can still show screenpipe through shortcut when shortcut is disabled

[onyedikachi-david]

Screen.Recording.2024-12-06.at.03.25.10.mp4

@louis030195 Seems to be the other way around for me.

[m13v]

is the feature to stop/start server? or specifically recording? when i press it my CLI stops

[m13v]

i can still show screenpipe through shortcut when shortcut is disabled

the same thing, it's just the text is mixed up, the toggle should be inverted

[m13v]

let's clean up warning, i see them when i build tauri app
logs.txt

[onyedikachi-david]

is the feature to stop/start server? or specifically recording? when i press it my CLI stops

Exactly to restart the server/CLI if it errors out or one just wants to stop and restart.

[onyedikachi-david]

Screen.Recording.2024-12-06.at.17.17.05.mov

how do you see this @louis030195 @m13v

[m13v]

i think the design is fine, is it supposed to restrart the server when u press it again?

[onyedikachi-david]

i think the design is fine, is it supposed to restart the server when u press it again?

Only when the server is off, one can use the shortcut to restart it.

[m13v]

does it restart for you? i don't thinking it reacts and the same for show overlay, if u press the short cut again should it be disabled?

…

On Fri, Dec 6, 2024 at 12:58 PM David Anyatonwu ***@***.***> wrote: i think the design is fine, is it supposed to restart the server when u press it again? Only when the server is off, one can use the shortcut to restart it. — Reply to this email directly, view it on GitHub <#846 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AY62CDB6T2HJKN5VMKZXSND2EIFY5AVCNFSM6AAAAABTBN7RPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRUGE3DINRTGE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[onyedikachi-david]

does it restart for you? i don't thinking it reacts and the same for show overlay, if u press the short cut again should it be disabled?
…
On Fri, Dec 6, 2024 at 12:58 PM David Anyatonwu @.> wrote: i think the design is fine, is it supposed to restart the server when u press it again? Only when the server is off, one can use the shortcut to restart it. — Reply to this email directly, view it on GitHub <#846 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AY62CDB6T2HJKN5VMKZXSND2EIFY5AVCNFSM6AAAAABTBN7RPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRUGE3DINRTGE . You are receiving this because you were mentioned.Message ID: @.>

When disabled, it doesn't react (as expected) to both shortcut trigger; but should when enabled.

[louis030195]

does this work now?

seems like very outdated now
kinda tired of testing and is not working