1021036 - Using grey default buttons is confusing

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect)

Description

[Adam Roach [:abr]]

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.

Comment 1

[Adam Roach [:abr]]

Attachment: Change default button color from grey to purple

Comment 2

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

Comment on attachment 8435021 [details] [diff] [review]
Change default button color from grey to purple

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

LGTM

Comment 3

[Adam Roach [:abr]]

https://hg.mozilla.org/projects/elm/rev/34856bd5bbcb

Comment 4

[Mark Banner (:standard8)]

https://hg.mozilla.org/mozilla-central/rev/34856bd5bbcb

Comment 5

[u279076]

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?

Comment 6

[Mark Banner (:standard8)]

(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).

Comment 7

[u279076]

(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.