Project

General

Profile

Actions

バグ #1527

closed

gtk3: MiraclePainter上でマウスを動かしたときにカーソル形状を適切に変更する が未対応

Added by Izumi Tsutsui about 1 month ago. Updated 12 days ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
gtk3
ブランチ:
topic/1527-miracle-painter-cursor-shape
クラッシュする:
No

Description

各種 gtk3 チケットが解決してきて mikutter_gtk3 がだいたい実用上問題なく動くようになってきていますが
いざリリースすると「gtk2 時代と違う」と言われそうな挙動をメモるチケットその2です。

gtk2 版では #1197 で「MiraclePainter上でマウスを動かしたときにカーソル形状を適切に変更する」が実装されていますが
gtk3 版では少なくとも見かけ上は未対応のはず。

#1197 と似たような差分で行けるのかな? という根拠のない予想はありますが、 gtk3 と cairo の仕様差次第?


Files

Actions #1

Updated by Izumi Tsutsui 30 days ago

なんとなく #1197863cf268 の差分と source:plugin/gtk3/widget/miraclepainter.rb@f06b9506 とを見比べて、
単に後者の gtk3 の最初のリビジョンから #1197 の一部がマージ漏れというか暫定あとまわし実装っぽいなー、
と思って適当に勘パッチしたら mikutter_gtk3-1527-poineter.mp4 の通りでそれっぽく動いているっぽいです。
window のスコープとかが野生の勘なので要レビュー)

Actions #2

Updated by Izumi Tsutsui 30 days ago

コンソール見てたら deprecated 出まくってたので修正してパッチ作り直しました。

{MIKUTTER_DIR}/plugin/gtk3/widget/miraclepainter.rb:698:in `set_cursor': 'Gdk::Cursor::HAND2' has been deprecated. Use 'Gdk::CursorType::HAND2' or ':hand2'.
{MIKUTTER_DIR}/plugin/gtk3/widget/miraclepainter.rb:700:in `set_cursor': 'Gdk::Cursor::XTERM' has been deprecated. Use 'Gdk::CursorType::XTERM' or ':xterm'.
{MIKUTTER_DIR}/plugin/gtk3/widget/miraclepainter.rb:702:in `set_cursor': 'Gdk::Cursor::LEFT_PTR' has been deprecated. Use 'Gdk::CursorType::LEFT_PTR' or ':left_ptr'.

Actions #3

Updated by Shibafu Midorino 25 days ago

私の環境だと eb5da23b6f50605c2d4dfc3e92ccd93f1c9d454d 時点で、パッチなしでカーソルが変更されているように見えます。

実質的な差異は指定しているカーソル名が若干変わっているだけだと思うのですが、なんでだろう…

Actions #4

Updated by Izumi Tsutsui 25 days ago

Shibafu Midorino さんは #note-3 で書きました:

私の環境だと eb5da23b6f50605c2d4dfc3e92ccd93f1c9d454d 時点で、パッチなしでカーソルが変更されているように見えます。

あれ? っと思って別の NetBSD/amd64 9.2 のノートPC環境で試したところ以下の動作でした。 gtk3設定によっては名前が異なるんですかね?
  • 通常エリアは 左上向け矢印(たぶん :left_pointer
  • メッセージ本文テキスト上だと I 文字形状の :xterm 相当になる
  • URL やハッシュタグ等のリンク文字列では デフォルト 左上矢印のままで このチケット添付の :hand2 のアイコンにはならない
    • パッチを当てるとリンク文字列で指差しマークの :hand2 になる
Actions #5

Updated by toshi_a 初音 12 days ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ set to topic/1527-miracle-painter-cursor-shape

パッチ参考に、よりシンプルな変更にしました。確認お願いします。

Actions #6

Updated by Izumi Tsutsui 12 days ago

  • Status changed from レビュー待ち to マージ待ち

toshi_a 初音 さんは #note-5 で書きました:

パッチ参考に、よりシンプルな変更にしました。確認お願いします。

topic/1527-miracle-painter-cursor-shape をマージしてリンク指印アイコン出ました。
OKと思います。

Actions #7

Updated by toshi_a 初音 12 days ago

  • Status changed from マージ待ち to 終了
Actions

Also available in: Atom PDF