Using grey default buttons is confusing

RESOLVED FIXED in mozilla33

Status

Hello (Loop)
Client
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: abr, Assigned: abr)

Tracking

unspecified
mozilla33
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The current CSS uses a grey color for the default button styling. This is confusingly similar to the color that many applications use to indicate that a control is inactive.
(Assignee)

Comment 1

4 years ago
Created attachment 8435021 [details] [diff] [review]
Change default button color from grey to purple
Attachment #8435021 - Flags: review?(nperriault)
Comment on attachment 8435021 [details] [diff] [review]
Change default button color from grey to purple

Review of attachment 8435021 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Attachment #8435021 - Flags: review?(nperriault) → review+
(Assignee)

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
The "invite someone to talk" button still appears grey to me in the latest Nightly. Does this bug refer to a different button or am I missing something?
Whiteboard: [qa?]
(In reply to Anthony Hughes, QA Mentor (:ashughes) from comment #5)
> The "invite someone to talk" button still appears grey to me in the latest
> Nightly. Does this bug refer to a different button or am I missing something?

This was back in the days when we had a button to click to get the call url, that has long since gone.

The toolbar icon is meant to be grey by default (https://people.mozilla.org/~dhenein/labs/loop-mvp-spec/#toolbar).
(In reply to Mark Banner (:standard8) from comment #6)
> The toolbar icon is meant to be grey by default
> (https://people.mozilla.org/~dhenein/labs/loop-mvp-spec/#toolbar).

Okay, thanks Mark.
Flags: qe-verify-
Whiteboard: [qa?]
You need to log in before you can comment on or make changes to this bug.