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

背景画像表示の不透明度を設定出来るように変更 #686

Closed
wants to merge 3 commits into from
Closed

背景画像表示の不透明度を設定出来るように変更 #686

wants to merge 3 commits into from

Conversation

beru
Copy link
Contributor

@beru beru commented Dec 8, 2018

背景画像表示の不透明度を設定出来るようにしました。
Issue #665 に対応しています。

不透明度は 0 が最小値で最大値は 255 です。0 の時は完全透明でその時には背景画像を描画しません。
この設定値は API 関数 AlphaBlend に渡す BLENDFUNCTION 構造体の SourceConstantAlpha メンバー(BYTE型)に指定する値です。

以下にタイプ別設定画面のスクリーンショットを貼り付けます。

日本語設定画面

image

英語設定画面

image

@beru beru added the enhancement ■機能追加 label Dec 8, 2018
@berryzplus
Copy link
Contributor

ユーザエクスペリエンスの観点から「透明度」を指定できるようにしたいです。

以下は、透明度を指定できる主要アプリの一つ、Excel 2016 の設定UIです。
厳密には Excel 365 なんですがややこしいので Excel 2016 だと思って見てください。

2018-12-09 2

UX観点からするとスピンコントロールだけでなくスライダーも付けるのが親切な感じがします。

透明度と不透明度は表と裏の関係にあります。
プログラムで使用するのが不透明度(例: 20%)、ユーザに設定させるのが透明度(例: 80%)みたいな感じ。

@arigayas
Copy link

arigayas commented Dec 9, 2018

image
Windows10のコマンドプロンプトでの透明度の指定では下限が30%に決められてますね。
完全に透明にするとウィンドウ操作が面倒くさい事になるからだと思われます。

@beru
Copy link
Contributor Author

beru commented Dec 9, 2018

image
Windows10のコマンドプロンプトでの透明度の指定では下限が30%に決められてますね。
完全に透明にするとウィンドウ操作が面倒くさい事になるからだと思われます。

おぉ、知らなかったです。試してみましたがコンソールの背景だけでなく全体が一様に透けてしまう方法ですね。これだと見辛くなっちゃいますね。。

なおこのPRでは背景画像を表示する際の不透明度の指定なので仮に 0 に指定しても背景画像を指定していないのと同じな事になります。なので下限にゲタを履かせる必要は無いと思います。

@beru
Copy link
Contributor Author

beru commented Dec 9, 2018

ユーザエクスペリエンスの観点から「透明度」を指定できるようにしたいです。

以下は、透明度を指定できる主要アプリの一つ、Excel 2016 の設定UIです。
厳密には Excel 365 なんですがややこしいので Excel 2016 だと思って見てください。

2018-12-09 2

UX観点からするとスピンコントロールだけでなくスライダーも付けるのが親切な感じがします。

透明度と不透明度は表と裏の関係にあります。
プログラムで使用するのが不透明度(例: 20%)、ユーザに設定させるのが透明度(例: 80%)みたいな感じ。

たしかに不透明度より透明度の方が分かりやすいですね。スライダーも付けた方が良いと思います。

なお百分率で指定する方が一般的に馴染みがあってユーザーにとって分かりやすいと思いますが、BYTE値に変換する必要が出てきてしまいます。そうすると厳密な数値指定が出来なくなるので 0 ~ 255 での指定する方法というのは個人的には保ちたいです。

Officeのライセンスも買い切り方式だけでなくSubscription方式が登場しましたね。まるでNHKの集金みたいで嫌ですが他のオフィススイートの使い勝手が悪いからしょうがないですね。そんな陰口を叩いたらお前が改善しろやと怖いペンギンとデーモンに恫喝されそうですが…。

@berryzplus
Copy link
Contributor

コマンドラインの設定オプションは「不透明度」なんですね・・・
同じ版元なのに設定項目名が違うとか、さすが大企業というか、統一感ねーな、というか(笑

項目名はサクラエディタの中でなんとなく統一できればいいと思ってます。
Excelよりはコマンドプロンプトに合わせるのがいいのかな?

不透明度の下限(=透明度の上限)は3割を設定しとくのがベターだと思います。
設定値が下回っていたら下限に切り上げで・・・

設定単位は [0, 255] でいいと思います。

確かに100分率のが馴染みやすいかも知れませんが、
サクラエディタユーザはこの設定の内部が [0, 255] で処理されることを知ってそうですし、
100分率だと微妙にズレて厳密な指定が難しくなるのは事実なので。

@berryzplus
Copy link
Contributor

どうでもいい系の豆知識。
[0, 255] は 0 ≦ x ≦ 255 を意味する表記なんだそうです。
[0, 256) なら 0 ≦ x < 256 の意味になるんだとか。
いちおうこれ数学記号なんだそうな・・・。

@beru
Copy link
Contributor Author

beru commented Dec 9, 2018

コマンドラインの設定オプションは「不透明度」なんですね・・・
同じ版元なのに設定項目名が違うとか、さすが大企業というか、統一感ねーな、というか(笑

項目名はサクラエディタの中でなんとなく統一できればいいと思ってます。
Excelよりはコマンドプロンプトに合わせるのがいいのかな?

透明度に変えてしまいました。まぁどっちでも良いといえばどっちでも良いんですが、コントロールのID名等に影響します。

不透明度の下限(=透明度の上限)は3割を設定しとくのがベターだと思います。
設定値が下回っていたら下限に切り上げで・・・

自分は変に下限上限を決めるのには反対です。3割がちょうど良いのかどうかというのも恣意的なものなので。まぁ決めの話に過ぎませんが、これについては特にそうしなくても大きな不都合は無いかと思います。もし本当に必要なのであれば説得お願いしますw

設定画面をOKを押して閉じて確定させないと画面表示には反映されないので設定しにくいとは思います。出来ればリアルタイムに確認出来るようにしたいですがまた別の機会にでも。

@berryzplus
Copy link
Contributor

透明度に変えてしまいました。まぁどっちでも良いといえばどっちでも良いんですが、コントロールのID名等に影響します。

了解っす。コントロールのIDは、色々変遷したのに初期の名前が残ってるやつとか結構あるので気にしなくていいかな、と思ってます。「左寄せ」オプションなので実処理が右寄せだったりすると、藁人形とトンカチを用意したくなりますが、まぁいいでしょう。というか、仮にそういう矛盾が発生したとすると、いまのPRレビューの枠組みをすり抜けられる気がしないです(笑

自分は変に下限上限を決めるのには反対です。3割がちょうど良いのかどうかというのも恣意的なものなので。まぁ決めの話に過ぎませんが、これについては特にそうしなくても大きな不都合は無いかと思います。もし本当に必要なのであれば説得お願いしますw

了解です。仮に透明度100%を指定した場合でも、
フレームウインドウは透けないので問題ない気がしてきました。

設定画面をOKを押して閉じて確定させないと画面表示には反映されないので設定しにくいとは思います。出来ればリアルタイムに確認出来るようにしたいですがまた別の機会にでも。

これはちょっち難しい気配です。主要部分の整理ができてから落ち着いて着手したい感じです。

@arigayas
Copy link

image
Windows10のコマンドプロンプトでの透明度の指定では下限が30%に決められてますね。
完全に透明にするとウィンドウ操作が面倒くさい事になるからだと思われます。

このissueは「背景画像についての設定」って事を忘れてました(苦笑)
なのでberuさんのコメントの

なので下限にゲタを履かせる必要は無いと思います。

に同意します。

@beru
Copy link
Contributor Author

beru commented Dec 12, 2018

設定画面の表記を 不透明度 から 透明度 に変更したのでスクリーンショットを再度貼り付けます。
スクリーンショットを取ろうとした時に英語の Transparency というラベルが見切れているのが分かったので修正しました。

設定画面(日本語)

image

設定画面(英語)

image

英語の Transparency に比べて日本語の 透明度 は横幅が短いのでちょっと空白が空いてしまいますね。

機能的には満たしているという認識なのでレビューお願いします。

@beru
Copy link
Contributor Author

beru commented Dec 14, 2018

しばらく待ちましたがレビューされなくて旬を過ぎてしまったのでボツにします。

@beru beru closed this Dec 14, 2018
@beru beru deleted the BackImg_opacity branch December 14, 2018 13:12
@ds14050
Copy link
Contributor

ds14050 commented Dec 14, 2018

短気ですねえ。代理 PR しちゃいますよ>master...ds14050:BackImg_opacity

@beru
Copy link
Contributor Author

beru commented Dec 14, 2018

問題無いですよ。Approve しておきます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ■機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants