機能 #1197
完了
MiraclePainter上でマウスを動かしたときにカーソル形状を適切に変更して欲しい
Yuto Tokunaga さんが6年以上前に追加.
6年以上前に更新.
説明
MiraclePainterにマウスオーバーした際に，リンク・テキスト・何もないところでそれぞれマウス形状を変更する機能の提案です．リンク上ではクリックして開けることを，テキスト上ではドラッグして選択できることをユーザーが把握できるという効果があります．以下の画像は機能のデモンストレーションです．
ファイル
- 期日 を 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させてもらいます。
カーソル形状を修正しました．あとカーソル形状の指定にCSSで規定される名前を使うように修正しました．
参考:
マウスカーソルを動かす度に`new`するのは無駄が多いと思うので，`MiraclePainter`初期化時に`Gdk::Cursor`を作成するよう変更したほうが良いかもしれません．
mikutterの他の箇所ではカーソルの形状を指定するのに名前は使ってなかったんですが、今はこれのほうがいいんですね。
インスタンスを持ち回すのはTooltipでも似たよなことをしていて、Tooltipはmikutter全体で一つのインスタンスを利用するような形になってるため、同じようなことをしようと思いました。
ただ、Gdk::Displayに依存するようになるので難しそうですね。
- ステータス を 実装待ち から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Yuto Tokunaga に変更
プロファイルなど取ったわけではないのですが、mikutterで他にマウスカーソルを変更しているところでも毎回 Gdk::Cursor.new を呼んでいるので、将来的に #1197-4 のようなアップデートをする可能性を考えて別のメソッドに処理を抽出しておきました。
本質的には何もやっていること変わっていませんが、一応これで確認してもらって問題ないようならmergeしようと思ってます。
topic/1197-miracle-cursor-shape ブランチを確認してもらって、問題ないようなら教えてください。
- 関連している 機能 #1203: Gdk::Cursorのインスタンスを得る方法を変更する を追加
#1197-3 の提案については、同じことをmikutterの中で何箇所もやる必要があるため、 #1203 の切り分けました。
他の形式にエクスポート: Atom
PDF
