Project

General

Profile

バグ #683

投稿欄にマウス中ボタンでペーストした場合に残り文字数表示が更新されない

Added by Izumi Tsutsui about 5 years ago. Updated about 5 years ago.

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

Description

バグ #675 と同じ話というか、その修正をテストして気づくべきだった話ですが、
#675 での「投稿ボタンが有効にならない」と同様の話で
マウス中ボタンによる文字ペーストのように paste_clipboard のシグナルが
発生しない場合やプラグインでの貼り付けなどキー入力でもない場合に、
投稿ボタンは有効になるようになりましたが残り文字数表示は更新されないようです。
(同じところで処理されてると思っててちゃんと見てませんでした……)

#675 での widget_post の修正と同じように widget_remain でも
'changed' のシグナルを引っ掛けるようにすると良いようです。
(添付パッチ参照)


Files

gtk_postbox.rb.diff (535 Bytes) gtk_postbox.rb.diff Izumi Tsutsui, 2014-05-25 04:55
0001-widget_post.ssc-key_release_event-widget_post.buffer.patch (1.18 KB) 0001-widget_post.ssc-key_release_event-widget_post.buffer.patch リビジョン 65699519 に対するパッチ Izumi Tsutsui, 2014-06-21 22:21

Related issues

Related to バグ #669: 特定の手順で操作するとPostboxが消える終了2014-05-09Actions

Associated revisions

Revision 2da32159 (diff)
Added by toshi_a 初音 about 5 years ago

投稿欄にマウス中ボタンでペーストした場合に残り文字数表示が更新されない refs #683

Revision ed74a9f1 (diff)
Added by toshi_a 初音 about 5 years ago

postboxに中ボタンクリックで文字列を貼り付けた時に字数カウントが更新されないので、字数カウントを更新するイベントを key_release_event から changed に変更 refs #683

Revision fd1b1dd8 (diff)
Added by toshi_a 初音 about 5 years ago

中ボタンクリックで貼り付けた時に残り文字数を更新する

History

#1

Updated by toshi_a 初音 about 5 years ago

確認してみます

#2

Updated by toshi_a 初音 about 5 years ago

  • Related to バグ #669: 特定の手順で操作するとPostboxが消える added
#3

Updated by toshi_a 初音 about 5 years ago

  • Status changed from 新規 to レビュー待ち
  • Assignee set to Izumi Tsutsui

ちょっと変えてますが対応してます。ただ、手持ちのトラックボールに中ボタンを割り当ててないので状況を再現できていません。そちらで動作しているか、念の為確認してください。

#4

Updated by Izumi Tsutsui about 5 years ago

  • Status changed from レビュー待ち to まだダメ

コミットされたものは

widget_post.ssc(:key_release_event){ |textview, event|

になっていますが、これだと中ボタンペーストでは反映されないです。

#675 と同じイベントなら

widget_post.buffer.ssc(:changed){ |textview, event|

でないといけないような。これだと中ボタンでも残り文字数反映されます。
(文法はよくわかってないです……)

#5

Updated by toshi_a 初音 about 5 years ago

  • Assignee changed from Izumi Tsutsui to toshi_a 初音

週末くらいにやってみます

#6

Updated by toshi_a 初音 about 5 years ago

  • Assignee changed from toshi_a 初音 to Izumi Tsutsui

言われたとおり修正したところ、mikutter起動時に必ずクラッシュするようになりました(リビジョン ed74a9f1)

/home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/glib2-2.2.0/lib/glib2/deprecatable.rb:85:in `signal_connect': no such signal: changed
     from /home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/glib2-2.2.0/lib/glib2/deprecatable.rb:85:in `block (3 levels) in extended'
     from /home/toshi/mikutter/core/mui/gtk_extension.rb:28:in `safety_signal_connect'
     from /home/toshi/mikutter/core/mui/gtk_postbox.rb:82:in `widget_remain'
     from /home/toshi/mikutter/core/mui/gtk_postbox.rb:46:in `generate_box'
     from /home/toshi/mikutter/core/mui/gtk_postbox.rb:40:in `initialize'
     from /home/toshi/mikutter/core/plugin/gtk/mikutter_window.rb:34:in `new'
     from /home/toshi/mikutter/core/plugin/gtk/mikutter_window.rb:34:in `add_postbox'
     from /home/toshi/mikutter/core/plugin/gtk/gtk.rb:282:in `block (2 levels) in <top (required)>'
     from /home/toshi/mikutter/core/event_listener.rb:25:in `call'
     from /home/toshi/mikutter/core/event_listener.rb:25:in `update'
     from /usr/lib/ruby/2.1.0/observer.rb:196:in `block in notify_observers'
     from /usr/lib/ruby/2.1.0/observer.rb:195:in `each'
     from /usr/lib/ruby/2.1.0/observer.rb:195:in `notify_observers'
     from /home/toshi/mikutter/core/event.rb:65:in `block (2 levels) in call'
     from /home/toshi/mikutter/core/event.rb:65:in `catch'
     from /home/toshi/mikutter/core/event.rb:65:in `block in call'
     from /home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/delayer-0.0.2/lib/delayer/procedure.rb:24:in `call'
     from /home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/delayer-0.0.2/lib/delayer/procedure.rb:24:in `run'
     from /home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/delayer-0.0.2/lib/delayer/extend.rb:58:in `run_once'
     from /home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/delayer-0.0.2/lib/delayer/extend.rb:30:in `run'
     from /home/toshi/mikutter/vendor/bundle/ruby/2.1.0/gems/delayer-0.0.2/lib/delayer.rb:43:in `method_missing'
     from /home/toshi/mikutter/core/plugin/gtk/delayer.rb:10:in `block in boot'
     from /home/toshi/mikutter/core/plugin/gtk/mainloop.rb:10:in `call'
     from /home/toshi/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
     from /home/toshi/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from /home/toshi/mikutter/mikutter.rb:63:in `boot!'
     from /home/toshi/mikutter/mikutter.rb:81:in `<main>'

お手数ですがもう一度パッチを作っていただけますか。

#7

Updated by Izumi Tsutsui about 5 years ago

git のブランチの使い方がよくわかっていなくて ed74a9f1 を取ってこれない('A`)のですが
widget_post.ssc
ではなくて
widget_post.buffer.ssc
でないといけないです。手打ちですいません。

#8

Updated by Izumi Tsutsui about 5 years ago

ようやくちゃんとパッチ作りました。(添付)
NetBSDのておくれLive Imageではこれと同じ中身のパッチ使ってます。

#9

Updated by toshi_a 初音 about 5 years ago

  • Status changed from まだダメ to レビュー待ち

ありがとうございます。適用しました

#10

Updated by Izumi Tsutsui about 5 years ago

  • Status changed from レビュー待ち to 解決

3.0.2 で確認しました。OKです。
どうもありがとうございます。

#11

Updated by toshi_a 初音 about 5 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF