プロジェクト

全般

プロフィール

バグ #1202

完了

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

Akira Ouchi さんが6年以上前に追加. 6年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
change_account.rb
ブランチ:
クラッシュする:
いいえ

説明

初めて特定のアカウントに切り替えるコマンドを使おうとしたら、うまく動作していないようでした。
前後のアカウントに移動するコマンドと同様に、: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


ファイル

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

toshi_a 初音 さんが6年以上前に更新

  • トラッカー機能 から バグ に変更
  • 開始日 を削除 (2018-04-06)
  • クラッシュするいいえ にセット

えっこマ?

toshi_a 初音 さんが6年以上前に更新

  • ステータス新規 から レビュー待ち に変更
  • 担当者Akira Ouchi にセット
  • ブランチtopic/1202-koma にセット

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

Akira Ouchi さんが6年以上前に更新

  • ファイル toolbar.png toolbar.png を追加
  • ステータスレビュー待ち から まだダメ に変更

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

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

toshi_a 初音 さんが6年以上前に更新

  • 担当者Akira Ouchi から toshi_a 初音 に変更

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

Akira Ouchi さんが6年以上前に更新

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

toshi_a 初音 さんが6年以上前に更新

  • ステータスまだダメ から レビュー待ち に変更
  • 担当者toshi_a 初音 から Akira Ouchi に変更

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

Akira Ouchi さんが6年以上前に更新

確認できました。

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

toshi_a 初音 さんが6年以上前に更新

  • ステータスマージ待ち から 終了 に変更

masterにmergeしました

他の形式にエクスポート: Atom PDF