プロジェクト

全般

プロフィール

バグ #993

ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開かれずにどの方法で開くか聞いてきます

ncaq net1年以上前に追加. 2日前に更新.

ステータス:
マージ待ち
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
ブランチ:
topic/993-fix-open-in-browser
クラッシュする:
いいえ

説明

ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開れずにどの方法で開くか聞いてきます.
選択肢は
  • ツイートの詳細を開く
  • 外部ブラウザで開く

です.

関連付けの設定は空で再現します.
関連付けをして聞かれないようにしようとしてもうまくいきませんでした.

ツイートに対して「詳細」を選ばずに「ブラウザで開く」を選択している時点で外部ブラウザで開きたいという意志は明確なので,聞かれずに外部ブラウザで開かれて欲しいです.

バグ #952 と多少重複していますが,こちらは「リンクを開く」では起きない現象のため,新規に開かせていただきました.

mikutterのコミット番号は d58ff9f581d69a2afa19dc1eb73070fb176ed947 システムはgentoo linuxです.

fixed-open-in-browser.patch (1.1 KB) fixed-open-in-browser.patch ncaq net, 2018-05-14 15:13

再現手順

ツイートに対してブラウザで開くコマンドを使う


関連するチケット

関連している バグ #952: ツイートを右クリックして出るメニュー中の "リンクを開く" や "ブラウザで開く" をクリックしたとき関連付けが無視される新規2016-12-20

関係しているリビジョン

リビジョン b1325bda (差分)
ncaq net11日前に追加

command: 「ブラウザで開く」コマンドのIntentの候補としてブラウザ以外がサジェストされていた refs #993

リビジョン bec42228 (差分)
toshi_a 初音11日前に追加

command: Webプラグインへの依存を解消 refs #993

履歴

#1 ncaq net1年以上前に更新

  • 題名ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開れずにどの方法で開くか聞いてきます から ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開かれずにどの方法で開くか聞いてきます に変更

#2 toshi_a 初音1年以上前に更新

確かに、open_in_browserコマンドは「ツイートの詳細」で開く必要はないですね。

vanillaの状態では、ブラウザで開かれるべきと思うのでそのような修正方針で行きます。

#3 toshi_a 初音1年以上前に更新

多分URLに対してIntentを醗酵しているのが問題で、Web Modelのインスタンスにすれば解決しそう。
世の中には内部ブラウザプラグインとかあるので( https://github.com/toshia/inBrowser )、これが入っている場合はIntent選択ダイアログが開き、候補として「外部ブラウザ」「内部ブラウザ」が表示されている必要がある。

#4 toshi_a 初音1年以上前に更新

  • 関連している バグ #952: ツイートを右クリックして出るメニュー中の "リンクを開く" や "ブラウザで開く" をクリックしたとき関連付けが無視される を追加

#5 ncaq net約1ヶ月前に更新

Web Modelを使うようにしました.
設定ダイアログで「デフォルトブラウザを使う」が設定されているとデフォルトブラウザ,chromiumなどのコマンドが設定されているとそれで開くようになります.
inBrowserはgem単独環境整えてないのでちょっと試せてないです.
inBrowserのintentがPlugin::Web::Webになってないのは良いのでしょうか…

#6 toshi_a 初音27日前に更新

  • ステータス新規 から パッチ適用待ち に変更

#7 toshi_a 初音11日前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者ncaq net にセット
  • ブランチtopic/993-fix-open-in-browser にセット

遅くなってしまいました。

パッチは一旦そのままmergeしました。
ただ、他のプラグインの定数(今回の場合Plugin::Web)は参照したくないので、 Diva::Model(:web) に書き換えています。
こちらで試しても問題ありませんでしたが、一応そちらでも確認してもらえますか。

#8 ncaq net2日前に更新

  • ステータスレビュー待ち から マージ待ち に変更

動作確認をしたのでステータスをマージ待ちに変更しました.

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