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

ユーザー辞書にしかない読みを変換しようとしてもできない問題を修正 #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ento
Copy link

@ento ento commented Feb 25, 2018

再現方法

  1. システム辞書に存在しない単語の読みを登録
  2. その読みを変換しようとスペースキーを押下すると、変換されるかわりに半角スペースが入力される

background.htmlのコンソールに出力されるエラー:

image

@hkurokawa
Copy link

@ento こんにちは。さいきん @jmuk さんにお願いしてこのレポジトリをforkさせてもらった黒川といいます。もしよければ、こちらの修正を自分のレポジトリに取り込みたいのですがPRを作っていただくことは可能でしょうか?いちばんご負担のない方法で取り込めたらと思います。よろしくお願いします。
https://github.com/hkurokawa/chrome-skk

@ento
Copy link
Author

ento commented Aug 8, 2022

@hkurokawa PR作りました: hkurokawa/chrome-skk#16 - コンフリクトもなく、簡単にできました。

ただ、 https://github.com/hkurokawa/chrome-skk/pulls にある緑の Create pull request ボタンを押すと、 jmukさんのレポジトリに対してPRを作る画面に飛ばされるので、もしフォークしたレポジトリの方が本家本元になるのであれば、頃合いをみてdetachするなどするといいかもですね。

@hkurokawa
Copy link

hkurokawa commented Aug 8, 2022

ありがとうございます!さっそくマージしまいた。

jmukさんのレポジトリに対してPRを作る画面に飛ばされるので

自分もこれは不便に感じていました。不勉強にして、detachができるのを知らなかったです。ただ、できたらjmuk/chrome-skkがfork元であることはどこかに残しておきたいので、PRのデフォルトのマージ先を変えられないか調べてみて、無理そうならdetachを検討してみようと思います。ありがとうございます!

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