-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
enhance(frontend): カラーモードの設定画面を視覚的にわかりやすく #13512
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13512 +/- ##
============================================
- Coverage 41.58% 14.27% -27.32%
============================================
Files 1617 779 -838
Lines 165038 71372 -93666
Branches 4072 1243 -2829
============================================
- Hits 68631 10188 -58443
+ Misses 95931 60761 -35170
+ Partials 476 423 -53 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This comment was marked as resolved.
This comment was marked as resolved.
c解消 |
同期設定に気付かずに設定が元に戻ると訴える人は、同期設定有効時に切り替えできなくしても「なぜか切り替わらない」という訴えに変わるだけな気もするわね |
とりあえずマージ |
あーでもacrylicPanelは用途が違う可能性がある |
なにかにオーバーレイするときに使う色な気がする(acrylicBgなど)ので使用用途としては間違ってない気はするけど |
これ使わないようにした |
あとdキーを押したときにデバイスカラーモードの追従も一緒に切るようにした |
ややオフトピですが、dキーのショートカットを意図せず押してしまって、何を押してどの設定が変わったのかわからない人がたくさん見受けられるので、ダイアログか通知だしてあげるなどしてあげるとより良い可能性があるなと思いました |
「次回以降表示しない」を実装するべきだが、それをここでやるとあれかもしれないので別PRか何かでまとめてやったほうが良い気がした |
「リノートしました。」と同じ形の表示でどうでしょう? |
「""何を押して""どの設定が変わったのかわからない」を解消しようと思うとメッセージが長くなりそう |
「何を押して」を諦めて「カラーモードを〇〇に変更しました」だけでも良いかもしれない |
postformやemojiPickerの入力欄からフォーカス外れているときにdを押して誤爆することがままあるので要らないならいっそ消しても良いかも |
デバッグ用途では使うのは私も(ほかソフトウェアなど作ってるものでは)頻出するのですが、日常使用で使いますかね |
朝はライトにして夜はダークにしてる |
(off-topic) デバイス同期でできないもんなのかしら |
なので1日2回は押すことになる |
Windowsにはそのような機能が無い |
これOSに機能なかったっけ(macOSにはあったと思うけどwindowsとかiosとかandroidだとないっけ) |
であれば適当な自動変更機能のほうが需要に近そう...? |
ダーク/ライトモードを時間指定で切り替えられるような機能追加を別PRですると良さそう |
Misskeyがそこまでケアするのは大げさじゃないかしら |
ナイトモード概念はあった気がしますが、ダーク/ライト切り替えはないかも。 |
とりあえずdひとおしで切り替えに関する賛否は一旦このPRのout of scopeになりそうなのでとりあえず discussion を立てますね |
This comment was marked as off-topic.
This comment was marked as off-topic.
=> #14067 |
@syuilo ショートカットキー周りの変更は一旦全部もとに戻した |
コンフリクト解消 |
コンフリクト解消 |
conflictを解消 |
コンフリクト解消 |
What
Why
Fix #13161
Additional info (optional)
Checklist