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
notification led for new message #4077
base: master
Are you sure you want to change the base?
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
Thank you! It is cool, but it will raise objections from some users as something that can compromise privacy - so there should be an option in Notification settings, something like "Show light" (and we can set default to "on" for the new users and "off" for the existing users). |
Couldn't it be possible to set the LED color per profile like in k9 email? And per profile enable/disable the notification led. Or should I open a separate issue for this? |
we don't set most privacy settings per profile, so no need to complicate it - the MVP to release it would be a global in-app setting that would apply to all profiles, because then it won't require any changes in the core - it just requires updating native preferences and adding a simple toggle in the UI. |
Fixes #4029