[MD Feedback] Add initial styling to feedback dialog.

[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.

BUG=640604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/85cfd05f49f4504cf5014cd1beb18841192b9328
Cr-Commit-Position: refs/heads/master@{#422472}

[apacible, 2016-09-04 23:00:33]

Description was changed from

==========
styles

BUG=
==========

to

==========
styles

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-09-04 23:01:55]

Description was changed from

==========
styles

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Feedback] Update style.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-09-05 05:30:28]

Description was changed from

==========
[MD Feedback] Update style.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Feedback] Add initial styling.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-09-05 05:34:32]

Description was changed from

==========
[MD Feedback] Add initial styling.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Feedback] Add initial styling.



CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-09-14 20:57:41]

Patchset #1 (id:1) has been deleted

[apacible, 2016-09-14 20:57:46]

Patchset #1 (id:20001) has been deleted

[apacible, 2016-09-14 20:59:47]

Description was changed from

==========
[MD Feedback] Add initial styling.



CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Feedback] Add initial styling to feedback dialog.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-09-14 21:12:39]

Description was changed from

==========
[MD Feedback] Add initial styling to feedback dialog.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[Needs Rebase] [MD Feedback] Add initial styling to feedback dialog.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-09-16 02:41:10]

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

[commit-bot: I haz the power, 2016-09-16 02:41:23]

Dry run: 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, 2016-09-16 03:47:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 03:47:51]

Dry run: This issue passed the CQ dry run.

[apacible, 2016-09-16 03:51:03]

Patchset #2 (id:60001) has been deleted

[apacible, 2016-09-16 03:51:09]

Patchset #1 (id:40001) has been deleted

[apacible, 2016-09-16 03:54:20]

Description was changed from

==========
[Needs Rebase] [MD Feedback] Add initial styling to feedback dialog.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Feedback] Add initial styling to feedback dialog.
==========

[apacible, 2016-09-16 03:54:20]

apacible@chromium.org changed reviewers:
+ afakhry@chromium.org

[apacible, 2016-09-16 04:07:40]

Description was changed from

==========
[MD Feedback] Add initial styling to feedback dialog.
==========

to

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.
==========

[apacible, 2016-09-16 04:09:08]

PTAL, thanks!

https://screenshot.googleplex.com/pQ7xk5am5XL.png

[afakhry, 2016-09-20 20:08:18]

On 2016/09/16 04:09:08, apacible wrote:
> PTAL, thanks!
> 
> https://screenshot.googleplex.com/pQ7xk5am5XL.png

Thanks! Looking at the screenshot, it seems you haven't yet incorporated the
changes in https://bugs.chromium.org/p/chromium/issues/detail?id=640604, right?
We should probably do that here too.

By the way, who decides the looks of this UI? Why are we still in this WIP stage
till now? It should be quite easy to finish the final UI in a single CL, but it
seems we're blocked on UX to finish the mocks?

[apacible, 2016-09-23 03:28:40]

> Thanks! Looking at the screenshot, it seems you haven't yet incorporated the
> changes in https://bugs.chromium.org/p/chromium/issues/detail?id=640604,
right?
> We should probably do that here too.
Right, I'll update that in the next patch.

> By the way, who decides the looks of this UI? Why are we still in this WIP
stage
> till now? It should be quite easy to finish the final UI in a single CL, but
it
> seems we're blocked on UX to finish the mocks?
We don't have a dedicated UXer yet, but am in the talks about how the feedback
dialog would fit into some of the ongoing desktop ux unification goals (happy to
chat offline about some of the discussion). This CL just makes it a bit nicer
while I finish wiring up the rest of the dialog.

[apacible, 2016-10-01 18:15:13]

Description was changed from

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.
==========

to

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.

BUG=640604
==========

[apacible, 2016-10-01 18:36:37]

Description was changed from

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.

BUG=640604
==========

to

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.

BUG=640604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[apacible, 2016-10-01 18:38:05]

Updated strings to reflect changes updates from 640604.

https://screenshot.googleplex.com/Ua2EgKxNDeB.png

[afakhry, 2016-10-03 17:39:37]

LGTM, thanks!

[apacible, 2016-10-03 17:50:15]

The CQ bit was checked by apacible@chromium.org

[commit-bot: I haz the power, 2016-10-03 17:51:23]

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, 2016-10-03 18:50:08]

Committed patchset #2 (id:100001)

[commit-bot: I haz the power, 2016-10-03 18:53:41]

Description was changed from

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.

BUG=640604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Feedback] Add initial styling to feedback dialog.

Small tweaks to current dialog, including text sizing and positioning.

BUG=640604
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/85cfd05f49f4504cf5014cd1beb18841192b9328
Cr-Commit-Position: refs/heads/master@{#422472}
==========

[commit-bot: I haz the power, 2016-10-03 18:53:42]

Patchset 2 (id:??) landed as
https://crrev.com/85cfd05f49f4504cf5014cd1beb18841192b9328
Cr-Commit-Position: refs/heads/master@{#422472}