Re: Review Request 63281: Clean up TaskList component layout.

2017-10-25 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63281/#review189268 --- Ship it! Master (5201cf1) is green with this patch.

Re: Review Request 63281: Clean up TaskList component layout.

2017-10-25 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63281/ --- (Updated Oct. 25, 2017, 8:08 p.m.) Review request for Aurora, Kai Huang and

Re: Review Request 63281: Clean up TaskList component layout.

2017-10-25 Thread Reza Motamedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63281/#review189228 --- Ship it! Ship It! - Reza Motamedi On Oct. 24, 2017, 11:32

Re: Review Request 63281: Clean up TaskList component layout.

2017-10-25 Thread Kai Huang
> On Oct. 25, 2017, 5:10 p.m., Kai Huang wrote: > > ui/src/main/js/components/TaskListItemActions.js > > Lines 3 (patched) > > > > > > s/TaskListItemActioins/TaskListItemHost/ ? > > David McLaughlin wrote: > The

Re: Review Request 63281: Clean up TaskList component layout.

2017-10-25 Thread David McLaughlin
> On Oct. 25, 2017, 5:10 p.m., Kai Huang wrote: > > ui/src/main/js/components/TaskListItemActions.js > > Lines 3 (patched) > > > > > > s/TaskListItemActioins/TaskListItemHost/ ? The idea is that we might add more

Re: Review Request 63281: Clean up TaskList component layout.

2017-10-25 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63281/#review189207 --- Ship it! LGTM

Re: Review Request 63281: Clean up TaskList component layout.

2017-10-24 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63281/#review189118 --- Ship it! Master (38476ab) is green with this patch.