Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: a way to turn off the name preview feature #372

Closed
smarter opened this issue Sep 9, 2020 · 6 comments
Closed

Feature request: a way to turn off the name preview feature #372

smarter opened this issue Sep 9, 2020 · 6 comments

Comments

@smarter
Copy link

smarter commented Sep 9, 2020

Recently, rikaichamp has started displaying results from the name dictionary by default (e8d61d8, 37007e1), I can see how that could be useful but I personally find it a bit too distracting and would appreciate an option for turning this off.

@melink14
Copy link

melink14 commented Sep 9, 2020 via email

@smarter
Copy link
Author

smarter commented Sep 9, 2020

I haven't used irikaichamp enough recently to say how common it is, but here's an example:
rikai

@smarter
Copy link
Author

smarter commented Sep 9, 2020

Maybe the name preview should only trigger if the matched string is written entirely in kanji?

@melink14
Copy link

melink14 commented Sep 9, 2020

Interesting example and heuristic.

I agree that would be right most of time though there are actually a lot of foreign place names that don't use Kanji of course (interested to see the false positive rate there...)

@nicolasmaia
Copy link

Maybe kanji and katakana can have a higher priority?

@birtles
Copy link
Member

birtles commented Sep 10, 2020

Yes, I'm in favor of tweaking the heuristic as opposed to adding a pref (most people will never find the pref, and the example you gave is likely to bother everyone, so we should fix it for everyone without making them turn off the name preview feature altogether).

I agree that the name entries that we want to surface in the preview are likely to be when we match on at least some kanji or katakana. Purely hiragana matches a more likely to be false positives. That seems like a good adjustment to the heuristic.

Other things we could do to mitigate the annoyance, apart from tweaking the heuristic are:

  • Reduce the number of kanji versions we show in the preview (e.g. just show 沖野、坂野、中堅… for the first example above). There is already a flag for this so it would be easy to tweak. That said, that sort of grouping only tends to happen when we match on hiragana anyway so maybe that's not needed.
  • Only show the first match plus … instead of up to 3 matches.

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

No branches or pull requests

4 participants