Project

General

Profile

Actions

機能 #1197

closed

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

Added by Yuto Tokunaga almost 7 years ago. Updated almost 7 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
Start date:
2018-04-01
Due date:
2018-04-10
% Done:

0%

プラグイン名:

Description

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


Files

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

Related issues

Related to 機能 #1203: Gdk::Cursorのインスタンスを得る方法を変更する新規2018-04-09

Actions
Actions #1

Updated by toshi_a 初音 almost 7 years ago

  • Due date set to 2018-04-10
  • Status changed from 新規 to 実装待ち
  • Assignee set to toshi_a 初音
  • Target version changed from 3.6 to 3.7

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

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

Actions #2

Updated by Yuto Tokunaga almost 7 years ago

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

参考:
Actions #3

Updated by Yuto Tokunaga almost 7 years ago

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

Actions #4

Updated by toshi_a 初音 almost 7 years ago

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

Actions #5

Updated by toshi_a 初音 almost 7 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Yuto Tokunaga

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

Actions #6

Updated by toshi_a 初音 almost 7 years ago

  • Related to 機能 #1203: Gdk::Cursorのインスタンスを得る方法を変更する added
Actions #7

Updated by toshi_a 初音 almost 7 years ago

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

Actions #8

Updated by Yuto Tokunaga almost 7 years ago

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

Actions #9

Updated by toshi_a 初音 almost 7 years ago

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

merged

Actions

Also available in: Atom PDF