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

Double entry for 冷 kanji. #499

Closed
Sindern opened this issue May 7, 2015 · 5 comments
Closed

Double entry for 冷 kanji. #499

Sindern opened this issue May 7, 2015 · 5 comments
Labels
bug

Comments

@Sindern
Copy link

@Sindern Sindern commented May 7, 2015

When looking for the 冷 kanji using handwriting recognition i noticed it's in the dictionary twice.

One gives results normally but the other returns no results.

Hopefully this is the right place to report this.

@mvysny mvysny added the bug label May 8, 2015
@mvysny
Copy link
Owner

@mvysny mvysny commented May 8, 2015

Yes, this is the right place, thanks for letting me know ;) I will verify this issue.

@mvysny
Copy link
Owner

@mvysny mvysny commented May 12, 2015

Failed to reproduce this issue with the built-in kanji pad. Are you perhaps using the KanjiPad paid extension?

@mvysny
Copy link
Owner

@mvysny mvysny commented May 12, 2015

Yes, reproduced on the KanjiPad paid extension.

@mvysny
Copy link
Owner

@mvysny mvysny commented May 13, 2015

Okay, there apparently are two similar characters:

  1. 冷 U+51B7 CJK UNIFIED IDEOGRAPH-51B7 cold, cool; lonely
  2. 冷 U+F92E CJK COMPATIBILITY IDEOGRAPH F92E cold, cool; lonely

Apparently, the first kanji is present in both Kanjidic and JMDict; second one is absent from both Kanjidic and JMDict. KanjiPad is correct to find both kanjis, perhaps Kanjidic should be updated to contain information about the second kanji. Can you please write to Jim Breen about your finding?
Meanwhile, I will add a notification message to Aedict, which will be shown in case when the kanji is missing from Kanjidic.

@mvysny
Copy link
Owner

@mvysny mvysny commented May 13, 2015

Implemented in Aedict 3.19

@mvysny mvysny closed this May 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.