バグ #993
closed
ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開かれずにどの方法で開くか聞いてきます
Description
選択肢は
- ツイートの詳細を開く
- 外部ブラウザで開く
です.
関連付けの設定は空で再現します.
関連付けをして聞かれないようにしようとしてもうまくいきませんでした.
ツイートに対して｢詳細｣を選ばずに｢ブラウザで開く｣を選択している時点で外部ブラウザで開きたいという意志は明確なので,聞かれずに外部ブラウザで開かれて欲しいです.
バグ #952 と多少重複していますが,こちらは｢リンクを開く｣では起きない現象のため,新規に開かせていただきました.
mikutterのコミット番号は d58ff9f581d69a2afa19dc1eb73070fb176ed947 システムはgentoo linuxです.
Files
再現手順
ツイートに対してブラウザで開くコマンドを使う
Related issues
Updated by ncaq エヌユル over 7 years ago
- Subject changed from ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開れずにどの方法で開くか聞いてきます to ツイートに対してブラウザで開く(open_in_browser)コマンドを使うと,ブラウザで開かれずにどの方法で開くか聞いてきます
Updated by toshi_a 初音 over 7 years ago
確かに、open_in_browserコマンドは「ツイートの詳細」で開く必要はないですね。
vanillaの状態では、ブラウザで開かれるべきと思うのでそのような修正方針で行きます。
Updated by toshi_a 初音 over 7 years ago
多分URLに対してIntentを醗酵しているのが問題で、Web Modelのインスタンスにすれば解決しそう。
世の中には内部ブラウザプラグインとかあるので( https://github.com/toshia/inBrowser )、これが入っている場合はIntent選択ダイアログが開き、候補として「外部ブラウザ」「内部ブラウザ」が表示されている必要がある。
Updated by toshi_a 初音 over 7 years ago
- Related to バグ #952: ツイートを右クリックして出るメニュー中の "リンクを開く" や "ブラウザで開く" をクリックしたとき関連付けが無視される added
Updated by ncaq エヌユル over 6 years ago
Web Modelを使うようにしました.
設定ダイアログで｢デフォルトブラウザを使う｣が設定されているとデフォルトブラウザ,chromiumなどのコマンドが設定されているとそれで開くようになります.
inBrowserはgem単独環境整えてないのでちょっと試せてないです.
inBrowserのintentがPlugin::Web::Webになってないのは良いのでしょうか…
Updated by toshi_a 初音 over 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee set to ncaq エヌユル
- ブランチ set to topic/993-fix-open-in-browser
遅くなってしまいました。
パッチは一旦そのままmergeしました。
ただ、他のプラグインの定数（今回の場合Plugin::Web）は参照したくないので、 Diva::Model(:web) に書き換えています。
こちらで試しても問題ありませんでしたが、一応そちらでも確認してもらえますか。
Updated by ncaq エヌユル over 6 years ago
- Status changed from レビュー待ち to マージ待ち
動作確認をしたのでステータスをマージ待ちに変更しました.