Project

General

Profile

Actions

バグ #1517

closed

MiraclePainter: 本文の文字選択ができない

Added by toshi_a 初音 over 3 years ago. Updated about 3 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
クラッシュする:
No

Description

MiraclePainterの文字列上でドラッグして文字列選択するとき、マウスボタンを離すと選択が解除される。
離したあと、更に右クリックして「選択範囲をコピー」などが実行できる必要がある。


Files

mikutter_gtk3_textselect.mp4 (49.9 KB) mikutter_gtk3_textselect.mp4 Izumi Tsutsui, 2021-10-23 17:11
2021-10-31 18.43.17.jpg (2.16 MB) 2021-10-31 18.43.17.jpg MiraclePainter::VERBOSE = trueにしてテキスト選択時のログを確認した図 Shibafu Midorino, 2021-10-31 19:21
Actions #1

Updated by Izumi Tsutsui over 3 years ago

おま環なのかなと思っていたのですが、こちらでも気になっていたので状況を書いておきます。

  • メッセージ(トゥート・ツイート)が事前に選択されていない状態でないと左クリックドラッグによる文字列選択ができない
    (メッセージを選択した後は左クリックドラッグで文字列選択できる)
  • 左クリックドラッグで文字列選択して左クリックを離すと文字列選択が解除される
    (チケット本文記載通り)
  • 選択された範囲の文字列が反転表示ではなく黒(文字色?)での塗りつぶしになる
  • 左クリックドラッグで文字列している最中に一度同時に右クリックしてから左クリックを離す、
    とすると文字列選択が残ったままで右クリックメニューでのコピーも可能
Actions #2

Updated by Shibafu Midorino about 3 years ago

MiraclePainterのVERBOSEフラグをオンにしてシグナルを追ってみました。
左クリックを離した直後、複数回 signal_do_state_flags_changed が呼ばれており、そのうち1回が selected フラグの無いものなのでこの時に外れてしまうようですね。

おそらく行自体の選択が外れた時に解除するためのものだと思うのですが、なんでこんなシグナルの飛び方をしているのかはいまいち分からず。

Actions #3

Updated by Shibafu Midorino about 3 years ago

  • Status changed from 分類待ち to 実装待ち
Actions #4

Updated by Shibafu Midorino about 3 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee set to toshi_a 初音
  • ブランチ set to topic/1517-mp-text-selection

選択範囲の色の修正と、勝手に解除される挙動を引き起こしているコードの削除を行ったら実用可能っぽい感じになってしまって困惑しています。
#note-2 で書いたようがおそらくやりたいであろう事をちゃんと実現すべきなのか、はてさて。

Actions #5

Updated by Izumi Tsutsui about 3 years ago

topic/1517-mp-text-selection をマージして動かしてみて、
#note-1 で書いた問題は一式解消していて意図通りに動いていると思います
  • メッセージを選択した状態でなくてもおもむろに左クリックドラッグしたら選択できる
  • 左クリックドラッグを離しても選択(反転)が維持される
  • 選択した部分は反転表示になる(白反転か背景色反転か見た目で区別できませんが)
  • (ついでに) URL 展開を含む文字列も意図通りコピーされる

マージ判断はとしぁさん見解待ちですかね。

Actions #6

Updated by toshi_a 初音 about 3 years ago

  • Status changed from レビュー待ち to 終了

merged.

Actions

Also available in: Atom PDF