feat: 画像縮小・非可逆圧縮の設定を分離

niri-la / misskey.niri.la

🌎 An interplanetary microblogging platform 🚀

GNU Affero General Public License v3.0

14 stars 1 forks source link

Closed i544c-me closed 6 months ago

i544c-me commented 7 months ago

cherry-picked from https://github.com/i544c-me/buicha-social-misskey/pull/3 この PR では、4択となっていた画像圧縮の設定を2つのトグルに分け、ユーザーがより直感的に設定できるよう修正を行いました。

Before

After

縮小の有無と非可逆圧縮の有無は分けて考えられますが、これを4択で提示してしまうとユーザーが混乱するのではないかと考えました。 2つのトグルならそれぞれ別に考えられますし、説明文も明確で分かりやすくなるかと思います。

Sayamame-beans commented 7 months ago

ありがとうございます！

to anatawa12

Sayamame-beans commented 7 months ago

「オリジナル画像を保持しない場合に、Web公開用画像の圧縮形式を選択できます。」の話がなくなるので、何の場合に作用する設定なのか若干分かりづらくなる可能性が少しありそうですかね…? どうやって対応しようかな

i544c-me commented 7 months ago

「オリジナル画像を保持しない場合に、Web公開用画像の圧縮形式を選択できます。」の話がなくなるので、何の場合に作用する設定なのか若干分かりづらくなる可能性が少しありそうですかね…? どうやって対応しようかな

そうなんですよね...「オリジナル画像を保持する」にチェックを入れた時は、圧縮形式の設定2つを見えなくしてしまうか、トグルを disabled の状態にするかしたら、分かりやすいかなとは思いました。

anatawa12 commented 7 months ago

そうなんですよね...「オリジナル画像を保持する」にチェックを入れた時は、圧縮形式の設定2つを見えなくしてしまうか、トグルを disabled の状態にするかしたら、分かりやすいかなとは思いました。

そうすると投稿画面でアップロード時にオリジナル画像を保持するをオフにした際の設定ができなくなるのでその実装では困ると思います

i544c-me commented 6 months ago

なるほど、そういえば投稿画面でも保持の設定ありましたね...

では設定画面に説明を加える方向が良いですかね...？こんな感じはいかがでしょう？

i544c-me commented 6 months ago

@Sayamame-beans ありがとうございます！修正取り込みました🙏

本家側で表記修正してもらったタイミングで直す

本家というのは misskey-dev/misskey ですかね？本家の方では 2560x2560 より小さくする説明自体無いような気がするのですが... 認識合っていれば、この PR で修正してしまって良いですかね...？

Sayamame-beans commented 6 months ago

本家の方では 2560x2560 より小さくする説明自体無いような気がするのですが...

確かにありませんでした(ごちゃまぜになってました、すみません) 修正助かります、お願いします🙏

i544c-me commented 6 months ago

修正しました！ご確認お願いします🙏