-
Notifications
You must be signed in to change notification settings - Fork 309
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
macOSにおいて最小化・最大化・閉じるボタンをタイトルバーの左側に表示する #558
macOSにおいて最小化・最大化・閉じるボタンをタイトルバーの左側に表示する #558
Conversation
#554 (comment) のような意見もいただきましたので、デザインについてはまだ様子を見ています…… #554 (comment) の最後の案も確かに自然だと思いました。 |
先ほどのコメント #554 (comment) にポジティブなリアクションがついたので、左上のボタンの背景が透過になるように変更しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Macは持ってないのでそこまでの動作確認はできませんが、他の2種を見る限り特に問題はなさそうです、LGTM
min-max-close-buttonsコンポーネントにまとめてしまうのはメンテナンス的にもよいと思いました
たしかに気になるといえば気になるので、簡単に消せそうなら消したいところかもです。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
内容
タイトルの通りです。 #554 の最初の案を採用しています。
関連 Issue
close #554
スクリーンショット・動画など
変更前
変更後
その他
Issue #554 に他のデザイン案もあります。より良い案が見つかればそちらに変更します。
手持ちのWindows環境がないので、macOS以外に動作確認を行ったのはLinuxのみです。
Vue.js や TypeScript に不慣れなので、ナンセンスなことをおこなっていた場合はご指摘よろしくお願いします。