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

Fix choseong converted into jongseong even TreatJongseongAsChoseong is off #530

Merged
merged 3 commits into from
Oct 3, 2021

Conversation

Riey
Copy link
Owner

@Riey Riey commented Oct 3, 2021

Summary

Fix #529

Note

Checklist

  • I have documented my changes properly to adequate places
  • I have updated the docs/CHANGELOG.md

Riey added 3 commits October 3, 2021 19:03
Choseong converted into jongseong even `TreatJongseongAsChoseong` is off
@Riey Riey added bug Something isn't working T-Engine labels Oct 3, 2021
@Riey Riey merged commit 23ba64a into develop Oct 3, 2021
@Riey Riey deleted the jongseong-bug branch October 3, 2021 10:17
@wind0204
Copy link
Contributor

wind0204 commented Oct 3, 2021

이 픽스를 적용하기 전에 FlexibleComposeOrder 애드온이 적용되어 있으면 #529 상황 제외하고 조합이 잘 되었습니다.

이 픽스를 적용하고 나서부터는 조합이 안 되는 경우가 많습니다.

ㅏ + 초성ㄱ + 종성ㅁ = 감
초성ㄱ + ㅏ + 종성ㅁ = 감
초성ㄱ + 종성ㅁ + ㅏ = ㅏ
ㅏ + 종성ㅁ + 초성ㄱ = 감
종성ㅁ + 초성ㄱ + ㅏ = ㅏ
종성ㅁ + ㅏ + 초성ㄱ = ㅁ가
감 + ㅏ + 초성ㄱ + 종성ㅁ = 감감
감 + 초성ㄱ + ㅏ + 종성ㅁ = 감감
감 + 초성ㄱ + 종성ㅁ + ㅏ = 감ㅏ
감 + ㅏ + 종성ㅁ + 초성ㄱ = 감감
*감 + 종성ㅁ + 초성ㄱ + ㅏ = 감ㅏ*
*감 + 종성ㅁ + ㅏ + 초성ㄱ = 감ㅁ가*

기존에는 위에 나열된 조합이 두개 빼고 다 잘 되어 '감감'이 출력되었지만, 마지막 두 조합에서 종성ㅁ 이 초성ㅁ 이 되어 '감ㅁ가', '감마ㄱ'가 출력되는 문제가 있었습니다.

릴리스가 이미 되서 아쉽군요... 핫픽스 배포를 대해 고려해보셔도 될듯 하네요.

@Riey
Copy link
Owner Author

Riey commented Oct 3, 2021

@wind0204 명세대로면 애드온을 켜도 중성이 초성앞에 올수있는게 전부고 종성의 순서를 바꾸진 못해야 합니다 문서에도 초성과 중성의 순서만 명시돼있고요

https://github.com/Riey/kime/blob/develop/docs/CONFIGURATION.ko.md#flexiblecomposeorder

따라서 감 + ㅏ + 종성ㅁ + 초성ㄱ가 감감이 되도록 원하신다면 신규 애드온이나 지금 애드온 기능의 변경이 필요하므로 새로 이슈를 열어주시면 감사하겠습니다

@wind0204
Copy link
Contributor

wind0204 commented Oct 3, 2021

@wind0204 명세대로면 애드온을 켜도 중성이 초성앞에 올수있는게 전부고 종성의 순서를 바꾸진 못해야 합니다 문서에도 초성과 중성의 순서만 명시돼있고요

https://github.com/Riey/kime/blob/develop/docs/CONFIGURATION.ko.md#flexiblecomposeorder

따라서 감 + ㅏ + 종성ㅁ + 초성ㄱ가 감감이 되도록 원하신다면 신규 애드온이나 지금 애드온 기능의 변경이 필요하므로 새로 이슈를 열어주시면 감사하겠습니다

일단 지금은 명세에 나오진 않았지만 예로 들어주신 감 + ㅏ + 종성ㅁ + 초성ㄱ가 감감이 잘 됩니다... 초성ㄱ + 종성ㅁ + ㅏ, 종성ㅁ + 초성ㄱ + ㅏ, 종성ㅁ + ㅏ + 초성ㄱ, 이 세가지가 '감'이 안 되는데, 이슈를 하나 열어보겠습니다. kime가 언젠가는 갖추어야할 기능같아서요...

@Riey
Copy link
Owner Author

Riey commented Oct 3, 2021

@wind0204 네 사실 되던게 버그(?)인셈인데 해당 기능은 원하시는 분들이 다수 있어서 이슈에서 좀 자세하게 다루면 좋겠네요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working T-Engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

세벌식 종성 입력 오류
2 participants