Skip to content

Conversation

@burntcouscous
Copy link

@burntcouscous burntcouscous commented Oct 3, 2025

Screenshot from 2025-10-03 19-35-18 Screenshot from 2025-10-03 19-35-09

It doesn't yet pause the timer though but functionally it should work leveraging your previous code of pausing and resuming.

BTW:

The main branch was already failing though; this is when I ran the version of Scribe I had already installed before I started working on the pause-resume branch:

image

So I couldn't test it all the way through.

@Mikodin
Copy link
Owner

Mikodin commented Oct 13, 2025

@burntcouscous I super appreciate you contributing!

Now that main is all good, I urge you to go further with it 😄

the handlePauseResumeRecording function doesn't exactly work, so wiring up buttons to use that function isn't enough.

I can't pretend to remember what was going on with it as it's almost a year ago, but there is a bug somewhere. I believe it had to do with moving between mobile and desktop

The roots are here
https://github.com/Mikodin/obsidian-scribe/blob/main/src/audioRecord/audioRecord.ts#L76-L82

@burntcouscous
Copy link
Author

Ok, I used Codex to make this change. I haven't tested it. Does it work?

@Mikodin
Copy link
Owner

Mikodin commented Nov 15, 2025

@burntcouscous briefly looking at the code, there is still no update for how the pause function works, this just added scaffolding around duration and displaying the correct time. I did not test it however

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.

2 participants