psethwick / plover_console_ui

Text User Interface plugin for Plover
GNU General Public License v3.0
20 stars 2 forks source link

Confusing old output in `dictionaries` menu #9

Closed user202729 closed 3 years ago

user202729 commented 3 years ago

Not really an issue, but this is confusing.

Example:

I see some options in this case.

  1. Print the dictionaries on every change by the user. (perhaps preferred)
  2. Print a status message ("Priority changed", etc.)
  3. Make a separate window for the dictionaries list.
psethwick commented 3 years ago

I also think the behaviour is weird. I have tried out option 1 already (which I also prefer) but having some async/thread interaction problems

Have also considered having a dictionaries window, which would honestly solve the async problem, but not sure I really love it.

Will try for option 1 still I think, just need to solve my race conditions somehow. Maybe I'll hook/unhook dictionaries_changed

psethwick commented 3 years ago

Was tripped up before by the dictionaries_loaded hook only being called once

Should work now, they do show up in config_changed, so I'm using that

psethwick commented 3 years ago

Fun side effect (which I'm counting as a feature):