Project

General

Profile

バグ #1202

特定のアカウントに切り替えるコマンドのWorld対応

Added by Akira Ouchi about 1 year ago. Updated about 1 year ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
change_account.rb
ブランチ:
topic/1202-koma
クラッシュする:
No

Description

初めて特定のアカウントに切り替えるコマンドを使おうとしたら、うまく動作していないようでした。
前後のアカウントに移動するコマンドと同様に、:world_change_currentをコールする必要があるのかと思い、
添付のパッチを作成してみました。

ただし、おそらくconditionが単にtrueなせいで、このパッチでは画面右下の設定タブにボタンが出てきてしまいます。
(visible:falseが無視される?)

特定のアカウントに切り替えるコマンドが追加されたコミットでもexecが空でしたが、動いてたんでしょうか…?

https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/69478c2a6a427ceec43ade366b49be6c8fffcc0f/diff/core/plugin/change_account/change_account.rb


Files

switch_account_to.diff (1.39 KB) switch_account_to.diff Akira Ouchi, 2018-04-06 23:17
toolbar.png (12.7 KB) toolbar.png Akira Ouchi, 2018-04-08 20:42
toolbar_visible.diff (738 Bytes) toolbar_visible.diff Akira Ouchi, 2018-04-10 22:28
akkiesoft@social.mikutter.hachune.net-.png (173 KB) akkiesoft@social.mikutter.hachune.net-.png Akira Ouchi, 2018-04-11 00:39

Associated revisions

Revision 0d3d0f7f (diff)
Added by Akira Ouchi about 1 year ago

特定のアカウントに切り替えるコマンドが実装されてなかった refs #1202

Revision 19812f1f (diff)
Added by toshi_a 初音 about 1 year ago

特定のアカウントに切り替えるコマンドの実装を短くした refs #1202

Revision 20417440 (diff)
Added by toshi_a 初音 about 1 year ago

gtk: windowロールコマンドのうち、visible属性がfalseのものがツールバーに表示されていた refs #1202

History

#1

Updated by toshi_a 初音 about 1 year ago

  • Tracker changed from 機能 to バグ
  • Start date deleted (2018-04-06)
  • クラッシュする set to No

えっこマ?

#2

Updated by toshi_a 初音 about 1 year ago

  • Status changed from 新規 to レビュー待ち
  • Assignee set to Akira Ouchi
  • ブランチ set to topic/1202-koma

マですね…。貰ったパッチから少し手を加えているので、一応そちらで動かしてみて意図通りになっているか確認してください

#3

Updated by Akira Ouchi about 1 year ago

world をそのままexecの中で使えたんですね(その発想はなかった…slugは使ったのにな?)。

右下にアイコンボタンが出てしまうのはとくに変わりないようですが、どうでしょうね。
もともとは存在していなかったのと、いちおうvisible:falseなはずなので、消えててほしい気もしています。

#4

Updated by toshi_a 初音 about 1 year ago

  • Assignee changed from Akira Ouchi to toshi_a 初音

あっ忘れてた
多分それは、ステータスバーのツールボタンの領域がvisibleフラグを見てないという問題だと思います。

#5

Updated by Akira Ouchi about 1 year ago

visibleを見るようにするパッチをつくってみました。

#6

Updated by toshi_a 初音 about 1 year ago

  • Status changed from まだダメ to レビュー待ち
  • Assignee changed from toshi_a 初音 to Akira Ouchi

ありがとうございます。 #1202-3 相当の内容を取り込んだので、これで大丈夫そうですね。問題無さそうならマージ待ちにしといてください

#7

Updated by Akira Ouchi about 1 year ago

確認できました。

なお、一部プラグインではvisibleがfalseになっているため、見えなくなってしまった場合はvisible:trueに変えてあげる必要がありそうです。
以下Mastodonでの会話ログ(ねんのため)

#8

Updated by toshi_a 初音 about 1 year ago

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

masterにmergeしました

Also available in: Atom PDF