Cleaned up the start page, added wait cursor, disabled Run button while waiting for results.

Cleaned up the start page, added wait cursor, disabled Run button while waiting for results.

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=14121

[mtklein, 2014-04-09 21:00:49]

lgtm

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/index.html (left):

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
experimental/webtry/templates/index.html:32: SkImageInfo info =
SkImageInfo::MakeN32(300, 300, kPremul_SkAlphaType);
I kinda liked these hints about what might be needed to run this yourself.  That
said, I'm just a humble code reviewer.  Take my opinions with a grain of salt.

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/index.html (right):

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
experimental/webtry/templates/index.html:28: <textarea name='code' id='code'
rows='15' cols='80'>SkPaint p;
Why not 24?  Heathen...

[jcgregorio, 2014-04-09 21:09:25]

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/index.html (left):

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
experimental/webtry/templates/index.html:32: SkImageInfo info =
SkImageInfo::MakeN32(300, 300, kPremul_SkAlphaType);
What if I add a download button that wraps the user code with all the other code
needed to compile and run it? 

I made these changes to clean up the UI, it was cluttered and at least on my
laptop the output and image was always below the fold.

On 2014/04/09 21:00:49, mtklein wrote:
> I kinda liked these hints about what might be needed to run this yourself. 
That
> said, I'm just a humble code reviewer.  Take my opinions with a grain of salt.

[mtklein, 2014-04-09 21:12:04]

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/index.html (left):

https://codereview.chromium.org/231853002/diff/1/experimental/webtry/template...
experimental/webtry/templates/index.html:32: SkImageInfo info =
SkImageInfo::MakeN32(300, 300, kPremul_SkAlphaType);
On 2014/04/09 21:09:26, jcgregorio wrote:
> What if I add a download button that wraps the user code with all the other
code
> needed to compile and run it? 
> 
> I made these changes to clean up the UI, it was cluttered and at least on my
> laptop the output and image was always below the fold.
> 
> On 2014/04/09 21:00:49, mtklein wrote:
> > I kinda liked these hints about what might be needed to run this yourself. 
> That
> > said, I'm just a humble code reviewer.  Take my opinions with a grain of
salt.
> 

Aw, I'm just being grumpy.  However you like is good by me.

Maybe start small and bring the wrapper code in with some snazzy CSS fade-in?

[jcgregorio, 2014-04-09 21:15:30]

The CQ bit was checked by jcgregorio@google.com

[commit-bot: I haz the power, 2014-04-09 21:15:33]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/jcgregorio@google.com/231853002/1

[commit-bot: I haz the power, 2014-04-09 22:54:04]

Change committed as 14121