Make the answers text color more consistent.

Make the answers text color more consistent.

All answer text (other than the suggestion text and positive/negative
text) is now dimmed.

BUG=709101
Review-Url: https://codereview.chromium.org/2798223004
Cr-Commit-Position: refs/heads/master@{#462651}
Committed: https://chromium.googlesource.com/chromium/src/+/93dc65041b3f7e4fe97ffc91b8ab3954a7a6bf6a

[Justin Donnelly, 2017-04-06 17:39:15]

Description was changed from

==========
Make all the answer text color more consistent.

All answer text (other than the suggestion text and positive/negative
text) is now dimmed.

BUG=709101
==========

to

==========
Make the answers text color more consistent.

All answer text (other than the suggestion text and positive/negative
text) is now dimmed.

BUG=709101
==========

[Justin Donnelly, 2017-04-06 17:40:01]

jdonnelly@chromium.org changed reviewers:
+ pkasting@chromium.org

[Justin Donnelly, 2017-04-06 19:29:51]

The CQ bit was checked by jdonnelly@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-06 19:30:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Peter Kasting, 2017-04-06 19:55:41]

My only caution here is that "dim" is supposed to mean "not as important to the
user, don't distract me".  It's not a physical markup thing that means "draw
this grey".  Be careful that semantically this is meaning what you want, rather
than just looking good.

If it doesn't mean the above, then maybe another fix (e.g. make more text black)
would achieve consistency in a better way.

That should be enough to let you make a good decision here, so LGTM

[Justin Donnelly, 2017-04-06 21:09:01]

On 2017/04/06 19:55:41, Peter Kasting wrote:
> My only caution here is that "dim" is supposed to mean "not as important to
the
> user, don't distract me".  It's not a physical markup thing that means "draw
> this grey".  Be careful that semantically this is meaning what you want,
rather
> than just looking good.
> 
> If it doesn't mean the above, then maybe another fix (e.g. make more text
black)
> would achieve consistency in a better way.
> 
> That should be enough to let you make a good decision here, so LGTM

Yes, that's the desired semantics.

[Justin Donnelly, 2017-04-06 21:16:30]

The CQ bit was unchecked by jdonnelly@chromium.org

[Justin Donnelly, 2017-04-06 21:16:34]

The CQ bit was checked by jdonnelly@chromium.org

[commit-bot: I haz the power, 2017-04-06 21:17:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-06 22:27:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491513394294360,
"parent_rev": "46076c58c1198abaac97db355103069c6ab2d2b2", "commit_rev":
"93dc65041b3f7e4fe97ffc91b8ab3954a7a6bf6a"}

[commit-bot: I haz the power, 2017-04-06 22:28:07]

Description was changed from

==========
Make the answers text color more consistent.

All answer text (other than the suggestion text and positive/negative
text) is now dimmed.

BUG=709101
==========

to

==========
Make the answers text color more consistent.

All answer text (other than the suggestion text and positive/negative
text) is now dimmed.

BUG=709101

Review-Url: https://codereview.chromium.org/2798223004
Cr-Commit-Position: refs/heads/master@{#462651}
Committed:
https://chromium.googlesource.com/chromium/src/+/93dc65041b3f7e4fe97ffc91b8ab...
==========

[commit-bot: I haz the power, 2017-04-06 22:28:08]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/93dc65041b3f7e4fe97ffc91b8ab...