fix(md-progress): set md-pixel-size on attached

[mattgaspar]

md-pixel-size was not working if set to a static value or set before the attached event. It was only working as used in the sample but didn't work as it would typically be used.

Also, let mdSize remain set when mdPixelSize is set. mdPixelSize overrides mdSize due to inline styles having higher priority.

[coveralls]

Coverage remained the same at 95.154% when pulling b7a83a7 on mattgaspar:master into 7bc059a on aurelia-ui-toolkits:master.

[Thanood]

@mattgaspar Thanks a lot. 👍

So, that empty bind() has a side-effect, is that right?
Any idea if that's intended from Aurelia's side? Looks a bit like "magic", tbh.

Also, let mdSize remain set when mdPixelSize is set. mdPixelSize overrides mdSize due to inline styles having higher priority.

Good catch, thanks! 😃

[mattgaspar]

It is intended behavior! I found a note in the docs yesterday when I was looking for a solution.
http://aurelia.io/hub.html#/doc/article/aurelia/framework/latest/creating-components/3

I thought I remembered tracking an issue when this was implemented but I couldn't find it. I just found many people reporting this as a bug!

It's definitely something to take note of. It seems to make sense, although I would have expected changed events to not fire on the initial bind in both scenario's.

P.S. This is my first pull request on GitHub! Let me know if I did anything wrong!

[Thanood]

If it's documented, we should be okay. 👍

P.S. This is my first pull request on GitHub!

In that case, congratulations. 😃