4 ms·
Awesome changes. One suggestion: In the Control Center™, I would recommend using the past-tense for the current state. E.g., Receive Notifications
by bhauer 10y ago
Awesome changes.
One suggestion: In the Control Center™, I would recommend using the past-tense for the current state. E.g.,
Receive Notifications Allowed X
Access Your Location Allowed X
Maintain Offline Storage Allowed X
As it exists in the screenshots, the present tense is used, and the X button seems to be associated with the word "Allow." Further clarification could be achieved by making the X button actually say "Disallow" and giving it a border separate from the word "Allowed." E.g.,
Receive Notifications Allowed [Disallow]
- toggle 10y ago> Further clarification could be achieved by making the X button actually say "Disallow" and giving it a border separate from the word "Allowed." I agree. An X usually means "close" or "hide this thing," whereas here they're using it to change a setting. It really looks like there should be a toggle switch there.
- past 10y agoThis is exactly right that X means "close" or "hide this thing" and this is what it does in this case as well: it removes the non-default setting and hides the list item.
- dao- 10y agoFiled https://bugzilla.mozilla.org/show_bug.cgi?id=1331172 https://bugzilla.mozilla.org/show_bug.cgi?id=1331172
- past 10y agoThanks for the suggestion, this has now been fixed.