Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scale/Crop実装 #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Scale/Crop実装 #21

wants to merge 1 commit into from

Conversation

gnz-hub
Copy link

@gnz-hub gnz-hub commented Feb 3, 2020

環境設定にScale/Crop追加。
CustomImageViewの描画時にサイズ等をごまかします。

諸問題
アプリケーション終了時に値は保存されません。
環境設定のキャンセル/OKに関わらず反映されます。
設定反映時に描画が乱れます。
ルーペの表示がズレます。
PDFのリンクもズレます。

@coo-ona
Copy link
Owner

coo-ona commented Feb 4, 2020

・本体に組み込むのであれば、諸問題の解決が必要です。
・scaleは「画面に合わせない」の時のみ動作するようですがそういう仕様でしょうか?
・値が保存されないのなら(保存する必要がないのであれば)環境設定ではなく、
 フィルターのようなユーティリティパネルにするべきかと思います。
 もしくは、
 環境設定で値を保存する(全体の設定)+ユーティリティパネルで本ごとの設定
 が理想かなぁと思います。

@gnz-hub
Copy link
Author

gnz-hub commented Feb 11, 2020

今回はとりあえずの機能提案かつ動作デモという感じでした。

おっしゃる通り値は保存される方が便利でしょう。
cropはユーティリティが適切そうです。

scaleは「画面に合わせないモード」のみの動作が仕様と考えてます。
他のフィットモードで動作すると「それもうフィットしてねーでしょ。」となりますので。
なので「scale」という独立した機能ではなく、

「画面に合わせない選択時に表示倍率を指定する YES/NO 倍率 xx%」

という項目を環境設定に追加するかたちですかね。
これで「画面に合わせないモード」が実質的に「フリースケールモード」になると。
この「フリースケールモード」こそ私が求めている機能なんです。
いや、正確にいうと真に欲しいのは「高さ指定モード」なんですが。

(実は私は以前cooViewerZoomというSIMBLPluginを作ってまして、
その当時から必要としていたのは「高さ指定機能」だったんです。
ですが名前をつけるにあたって適当なものが思いつかなくてですね、
「zoomにするとooが対称になっていい感じだな、よし倍率設定がメインの体でいこう。」
という経緯があったのです。)

ただ、改めて考えてみても「高さ指定モード」って、
cooViewerにはそぐわないというか立ち位置がないように思うんです。
なので「フリースケールモード」ならまだ導入される余地があるかなと。

問題点に関しては先に挙げたもの以外にも色々とありそうなので、
さらに検証してみます。

@coo-ona
Copy link
Owner

coo-ona commented Feb 13, 2020

フリースケールの件、
UIとしてはメニューの表示=>フリースケールを追加して、
選択されたら倍率を入力とかが良いかなぁと思います。
(というか私も以前検討していました。自分的には不要だったので後回しになってましたが。。)

@gnz-hub
Copy link
Author

gnz-hub commented Feb 20, 2020

新規にフリースケールモードを設けるかたちですかね。
それでいってみましょう。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants