-
Notifications
You must be signed in to change notification settings - Fork 167
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
シングルクォートで囲まれたメールアドレスの選択 #1260
Comments
この事象は分からんです。
対策としては、メールアドレスの終端の次の文字がメールアドレスの先頭文字と一致する場合、先頭文字をメールアドレスに含めない、というようなコードを書くことになりそうです。 この周辺は検出ロジックが超低レベルであること、レイアウト・ロジック単位の取り違え問題が関連することなどから、修正とレビューの難易度が非常に高い気がします。 間違ってたらすまん!でどんどん進めちゃうのも一つの作戦かもしんないです・・・。 |
いつからか正確には分かりませんが ver.2.2.01 では正しく動作していました。 |
#792 の影響ですかね。 |
検証しようと思って画像よく見たら先頭のシングルクォートに下線が付いてますね。 過去版 v2.3.2 でも試しましたが同じ現象でした。 つまり、GitHub移行後に壊したわけではない 。。。(  ̄Д ̄)ヨッシャ!! どうも2016年頃に「記号類が含まれるメールアドレス」を検出できるようにしたらしくって、その時の変更の影響っぽい気がします。責任逃れしてもしゃあないんですけど(笑)。 たまたま、いまGWですし、今年のGWは全世界的に |
WikipediaやRFCなどを見ても、シングルクォート自体は、ローカルパート(ユーザー名)のところに直書きしていい文字として指定されているので、これはちょっと複雑な問題だと思います。 下のリストをこのissueeの表示と同じような感じにしたいだけなら、ローカルパートの許容文字リストwcschr?からシングルクォートを外せばいいんじゃないでしょうか。 |
これのワークアラウンド。 sakura-editor#1260
これのワークアラウンド。 sakura-editor#1260
問題内容
シングルクォートで囲まれたメールアドレスをクリックすると先頭のシングルクォートまで選択されしまう。末尾のシングルクォートは選択されない。本来はメールアドレスの部分だけ選択されるべき。
再現手順
シングルクォートで囲まれたメールアドレスが表示されている状態でメールアドレスの部分をクリックするとメールアドレスと共に先頭のシングルクォートまで選択されてしまう。
再現頻度
100%
問題のカテゴリ
環境情報
Windows10 Pro 64bit
v2.4.0.2686 32bit (tag v2.4.0)
スクリーンショット
The text was updated successfully, but these errors were encountered: