FooSoft / yomichan-anki

Plugin for sentence/vocab mining Japanese books in Anki.
https://foosoft.net/projects/yomichan-anki
Other
58 stars 12 forks source link

Usability proposals #4

Closed felixvd closed 9 years ago

felixvd commented 9 years ago

I just picked this up and thought that these things could make the first uses a bit easier. I'll just drop these as comments:

Lastly, a digital currency donation address would be appropriate somewhere :)

Thanks!

FooSoft commented 9 years ago

These are good ideas. About having the text box editable -- do you want to actually be able to edit it, or is being able to paste text to it sufficient? I don't want to create a situation where users can accidentally make edits to the text file they are reading, if you know what I mean.

felixvd commented 9 years ago

I think I'd do both. Editing the field would be useful for treating notes I take during presentations or classes. My workflow at the moment involves a browser window, Rikaichan and lots of copy-pasting.

You are right in saying that overwriting an opened file would be unintuitive and might be destructive. As a user I wouldn't expect the text box to save edits to a file, unless it's a temp file to keep the box persistent between sessions. If it worked like that, making the field completely editable would pose no risk, I think.

Afterwards, a separate save option for the field may be desirable to some, but I wouldn't sweat it.

Thanks for the response, looking forward to your changes :)

felixvd commented 9 years ago

I might have misunderstood what you meant - if the text box is editable you might edit it by accident, yes. Most users will know to undo via hotkey, and for the rest you could put in an Undo button. Making the box lockable is another option, but I'd vote for leaving it editable by default - I wouldn't think to look for a button to unlock it as a new user.

Cheers!

FooSoft commented 9 years ago

All done with the exception of having tool tips for the buttons. The HTML widget in Qt4 is primitive and doesn't allow this to be done through HTML : (

New version pushed out.