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

feat: canBeJungseong 함수가 분리되지 않은 이중모음도 중성으로 인식하도록 개선합니다. #333

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

shren207
Copy link

@shren207 shren207 commented Mar 15, 2025

Overview

canBeJungseong 함수가 인수로 ,,,를 제외한 이중모음(이하 복합모음)을 받을 때, 모음이 분리되어 있지 않다면 이를 중성으로 인식하지 않습니다.

canBeJungseong('ㅗㅏ') // true
canBeJungseong('ㅘ')  // false

따라서 복합모음을 canBeJungseong의 인수로 넘겨줄 때는 모음을 분리해서 넘겨줘야 하는데, 다음과 같은 이유로 다소 불편하다고 느껴졌습니다.


1. 복합 모음을 분리해서 인수로 넘겨주는 방식이 직관적이지 않음

ㅗㅏ와 같이 분리된 형태보다는 와 같이 합쳐진 형태로 인수를 넘기는 것이 더 직관적이라고 생각됩니다.

2. 키보드에서 복합모음을 분리해서 작성하는 것이 불편

맥OS와 윈도우는 복합모음 입력 시 다음과 같이 자동으로 분리되어 있는 모음을 하나로 합쳐줍니다.

composite_vowels

합쳐지는 것을 막으려면 아래 영상과 같이 첫 모음 입력 후 다음 모음을 입력하기 전에 한 칸 띄워줘야 하는 불편함이 있습니다.

output


따라서 복합모음 또한 종성으로 인식할 수 있도록, 배열 JUNSEONGS에 값을 추가하였습니다.

canBeJungseong('ㅗㅏ') // true
canBeJungseong('ㅘ')  // true

PR Checklist

  • I read and included theses actions below
  1. I have read the Contributing Guide
  2. I have written documents and tests, if needed.

Copy link

changeset-bot bot commented Mar 15, 2025

⚠️ No Changeset found

Latest commit: 619f6a0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Mar 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
es-hangul ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 30, 2025 9:19am

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

Successfully merging this pull request may close these issues.

1 participant