Project

General

Profile

Actions

バグ #1202

closed

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

Added by Akira Ouchi almost 7 years ago. Updated almost 7 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
change_account.rb
ブランチ:
クラッシュする:
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
Actions #1

Updated by toshi_a 初音 almost 7 years ago

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

えっこマ?

Actions #2

Updated by toshi_a 初音 almost 7 years ago

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

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

Actions #3

Updated by Akira Ouchi almost 7 years ago

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

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

Actions #4

Updated by toshi_a 初音 almost 7 years ago

  • Assignee changed from Akira Ouchi to toshi_a 初音

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

Actions #5

Updated by Akira Ouchi almost 7 years ago

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

Actions #6

Updated by toshi_a 初音 almost 7 years ago

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

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

Actions #7

Updated by Akira Ouchi almost 7 years ago

確認できました。

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

Actions #8

Updated by toshi_a 初音 almost 7 years ago

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

masterにmergeしました

Actions

Also available in: Atom PDF