Project

General

Profile

機能 #977

.jpgまたは.pngで終わるファイル名の画像は、サードパーティプラグインが開くルールを設定しても、標準のルールに上書きされてしまう

Added by あひる 家鴨 over 3 years ago. Updated over 3 years ago.

Status:
終了
Priority:
通常
Target version:
Start date:
2017-01-24
Due date:
% Done:

0%

プラグイン名:
openimg

Description

現状画像の直リンクは、mikutterのopenimgが定義しています。
プラグイン側でdefimageopenerを定義し、画像をその定義に基づいて開こうとしてもopenimgの定義が適用されてしまっているようです。
slackだと画像を開くのに、リクエストヘッダーにトークンを入れるなどの作業が必要なため、openimgの定義では開けません。
プラグイン側で画像の直リンクを開く部分を定義できるようになるといいと思いますが、どうでしょうか。


Related issues

Related to 機能 #1009: Wikipediaの画像を開く新規toshi_a 初音2017-03-27

Actions
#1

Updated by toshi_a 初音 over 3 years ago

  • Subject changed from openimgが行なっている画像直リンクの機能をプラグインにも持たせたい to .jpgまたは.pngで終わるファイル名の画像は、サードパーティプラグインが開くルールを設定しても、標準のルールに上書きされてしまう
  • Assignee set to toshi_a 初音
  • プラグイン名 changed from core/plugin/openimg/openimg.rb to openimg

Wikipediaでも同様の問題が起こっています。拡張子でマッチするのは開く方法がわからない時のフォールバック手段なので、ルールとしては定義しない方針で行こうと思います

#2

Updated by toshi_a 初音 over 3 years ago

  • Status changed from 新規 to 実装待ち

ルールから「画像直リンク」を消して、 openimg_raw_image_from_display_url フィルタに、ルールを見つけられない時は画像をダウンロードするような変更を行ったところ、サードパーティプラグイン等がPhoto Modelを利用せずURLを直接指定して画像を開こうとした場合に、「画像ビューア」インテントがサジェストされず、ブラウザで開いてしまうようになりました。

3.5の間に上記のような副作用のある変更は行なえませんが、3.6でもできれば上記のような状態のときに画像ビューアがサジェストされるようにしたいので、ちょっと考えます。

具体的に決まっていないのでバージョンは変えませんが、どの方法を取っても大きめの変更になりそうなので、3.6を対象とすると思います。

#3

Updated by toshi_a 初音 over 3 years ago

  • Related to 機能 #1009: Wikipediaの画像を開く added
#4

Updated by toshi_a 初音 over 3 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to あひる 家鴨
  • Target version changed from 3.5 to 3.6

結局考えた結果、これで行こうかなと思います(二週間も熟成させてしまった)。

topic/977-openimg-fallback にpushしたので、レビューお願いします。
#977#note-2 のとおり、互換性に配慮して3.6からとしたいので、もしこれで問題がなければ develop にmergeしてください。

#5

Updated by あひる 家鴨 over 3 years ago

  • Status changed from レビュー待ち to 終了

slack プラグインで defimageopener を定義したところ、そちらが適用されているのを確認できました。
develop にマージしました。
ありがとうございます。

Also available in: Atom PDF