nikdanilov / whisper-obsidian-plugin

Speech-to-text in Obsidian using OpenAI Whisper
MIT License
227 stars 31 forks source link

"Sending/parsing" popups on mobile cover significant portion of screen #24

Closed maxwelljoslyn closed 10 months ago

maxwelljoslyn commented 1 year ago

In the desktop version of Obsidian, there are notification pop-ups which give feedback that the Whisper plugin is operating correctly.

On desktop, these do not get in the way. They are just small bubbles in the top right of Obsidian. However, on mobile, the large, bubble-shaped notifications take up about 1/3 of the visible typing area above the keyboard. This means that I cannot see any the text that I am editing, or that the plugin has just inserted, if it falls in that area. Since I use the Whisper plugin every 10 or 15 seconds to create more text, this is a significant barrier to my use of the plugin on mobile.

Would it be possible to add one or more of the following options to the plugin?

Nik D, thank you for all this work on this plugin. As someone suffering from RSI that limits my ability to type, Whisper is the secret weapon which will allow me to turn in my master's thesis on time. Your plugin is the easiest interface to Whisper that I have yet tried, and has replaced almost entirely my use of the extremely crappy iPhone dictation, which has been a huge struggle for academic and professional use the last three years.

maxwelljoslyn commented 1 year ago

Here are some screenshots showing the extent of the problem.

Before Triggering Plugin, or While Recording With It

Note: the question mark icon at the top left of the ribbon above the keyboard is the button to trigger Whisper recording thru the plugin.

obsidian-whisper-1

First Two Popups

obsidian-whisper-2

Third Popup

obsidian-whisper-3

Popups Disappear

After about two or three seconds, the popups disappear and I can finally see what I "typed."

obsidian-whisper-4

nikdanilov commented 1 year ago

I am thinking now that success notifications do not provide any value. They can be completely removed.

maxwelljoslyn commented 1 year ago

Thanks! By success notification you mean the third of the three? If so I agree.

Hopefully you don’t want to remove all, though. As a user, I like the first notification: 1) it tells me that the extension actually triggered 2) seeing the size of the audio that got sent is a nice sanity check for the length of the audio.

For what it’s worth, I think only the first notification needs to be user facing and the second one could be behind an option or saved for developer debugging. But maybe there’s a use case I’m not seeing for having that one visible. May I ask what its purpose is?

On Sat, Jun 17, 2023 at 23:33 Nik Danilov @.***> wrote:

I am thinking now that success notifications do not provide any value. They can be completely removed.

— Reply to this email directly, view it on GitHub https://github.com/nikdanilov/whisper-obsidian-plugin/issues/24#issuecomment-1595986935, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACY2B6PYTJZTJILHOFVT233XL2OMXANCNFSM6AAAAAAZHKUVMM . You are receiving this because you authored the thread.Message ID: @.***>

nikdanilov commented 1 year ago

Right. I concur that the first alert could be the only one displayed to users and the second one behind an optional debugging setting. The initial purpose was to ensure that the process was running smoothly and keep track of each stage of the operation. Will update.

maxwelljoslyn commented 1 year ago

Excellent, thanks for your responsiveness.

On Mon, Jun 19, 2023 at 07:38 Nik Danilov @.***> wrote:

Right. I concur that the first alert could be the only one displayed to users and the second one behind an optional debugging setting. The initial purpose was to ensure that the process was running smoothly and keep track of each stage of the operation. Will update.

— Reply to this email directly, view it on GitHub https://github.com/nikdanilov/whisper-obsidian-plugin/issues/24#issuecomment-1597307178, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACY2B6NQ6RCFUA5227HE6QDXMBP7NANCNFSM6AAAAAAZHKUVMM . You are receiving this because you authored the thread.Message ID: @.***>

nikdanilov commented 10 months ago

Fixed by hiding the sending and parsing popups behind the debug mode.