プロジェクト

全般

プロフィール

機能 #1197

MiraclePainter上でマウスを動かしたときにカーソル形状を適切に変更して欲しい

Yuto Tokunaga22日前に追加. 9日前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2018-04-01
期日:
2018-04-10
進捗率:

0%

プラグイン名:

説明

MiraclePainterにマウスオーバーした際に,リンク・テキスト・何もないところでそれぞれマウス形状を変更する機能の提案です.リンク上ではクリックして開けることを,テキスト上ではドラッグして選択できることをユーザーが把握できるという効果があります.以下の画像は機能のデモンストレーションです.

1197.patch (1.35 KB) 1197.patch Yuto Tokunaga, 2018-04-01 15:17
1197-2.patch (1.54 KB) 1197-2.patch Yuto Tokunaga, 2018-04-07 01:42

関連するチケット

関連している 機能 #1203: Gdk::Cursorのインスタンスを得る方法を変更する新規2018-04-09

関係しているリビジョン

リビジョン 29515ab9 (差分)
Yuto Tokunaga18日前に追加

MiraclePainter上でマウスを動かしたときにカーソル形状を適切に変更する refs #1197

リビジョン ba9442e1 (差分)
Yuto Tokunaga14日前に追加

MiraclePainterのカーソルの形状が誤っていた refs #1197

リビジョン 863cf268 (差分)
toshi_a 初音14日前に追加

MiraclePainterのマウスカーソルを変更する処理を一箇所にまとめた refs #1197

履歴

#1 toshi_a 初音18日前に更新

  • 期日2018-04-10 にセット
  • ステータス新規 から 実装待ち に変更
  • 担当者toshi_a 初音 にセット
  • 対象バージョン3.6 から 3.7 に変更

これとても良いですね。基本的にはこのままmergeしたいと思うので、ブランチ topic/1197-miracle-cursor-shape に 1197.patch をpushしました。
新機能なので、3.7でリリースするため、developブランチにmergeします。

ただ、うちの環境(i3wmを使っている)では、HAND1が指差している手ではなく、手でつかむマークになっています。マウスカーソルの形状のコントロールはよく分かってないので、休日に調べるか、4/10を過ぎても時間が取れない場合はこのままdevelopにmergeさせてもらいます。

#2 Yuto Tokunaga16日前に更新

カーソル形状を修正しました.あとカーソル形状の指定にCSSで規定される名前を使うように修正しました.

参考:

#3 Yuto Tokunaga16日前に更新

マウスカーソルを動かす度に`new`するのは無駄が多いと思うので,`MiraclePainter`初期化時に`Gdk::Cursor`を作成するよう変更したほうが良いかもしれません.

#4 toshi_a 初音14日前に更新

mikutterの他の箇所ではカーソルの形状を指定するのに名前は使ってなかったんですが、今はこれのほうがいいんですね。
インスタンスを持ち回すのはTooltipでも似たよなことをしていて、Tooltipはmikutter全体で一つのインスタンスを利用するような形になってるため、同じようなことをしようと思いました。
ただ、Gdk::Displayに依存するようになるので難しそうですね。

#5 toshi_a 初音14日前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Yuto Tokunaga に変更

プロファイルなど取ったわけではないのですが、mikutterで他にマウスカーソルを変更しているところでも毎回 Gdk::Cursor.new を呼んでいるので、将来的に #1197-4 のようなアップデートをする可能性を考えて別のメソッドに処理を抽出しておきました。
本質的には何もやっていること変わっていませんが、一応これで確認してもらって問題ないようならmergeしようと思ってます。
topic/1197-miracle-cursor-shape ブランチを確認してもらって、問題ないようなら教えてください。

#6 toshi_a 初音14日前に更新

  • 関連している 機能 #1203: Gdk::Cursorのインスタンスを得る方法を変更する を追加

#7 toshi_a 初音14日前に更新

#1197-3 の提案については、同じことをmikutterの中で何箇所もやる必要があるため、 #1203 の切り分けました。

#8 Yuto Tokunaga9日前に更新

確認しました.問題ないです.

#9 toshi_a 初音9日前に更新

  • ステータスレビュー待ち から 終了 に変更

merged

他の形式にエクスポート: Atom PDF