On Thu, 2011-06-16 at 22:47 +0200, Piotr Przybylski wrote:
2011/6/16 Rouslan Placella rouslan@placella.com:
On Thu, 2011-06-16 at 21:05 +0200, Piotr Przybylski wrote:
2011/6/16 Rouslan Placella rouslan@placella.com:
Hi there,
I've added some animations to the routines functionalities in my branch and I'm just looking for your opinions about this. A demo video is available on my site [1]. Please let me know if you think that it's too much eye-candy or if you think that it's OK. I didn't check this into my repo yet and I'm keeping the code in my git-stash.
Personally, I don't like the table jumping up and down. Moving objects attract attention, especially if a large part of the screen moves, and in this case it looks like an effect done because it can be done. Space above Routines table should expand / contract to proper height in one move - it it needs two, then the effect stops looking nice, it becomes annoying.
If it can't be done, it would look nice with old content replaced by new with 0 opacity and using fade in to show it.
(Note - I can be prejudiced, I hate collapsible options in PMA because the animation used there is just wrong imo)
Well, I added in the animations because without them the content on the page just flashes and I find that really disturbing, not just because I could (you can see this functionalities without the animations on my demo server). But yeah, you have a good idea there about making the content slide once from the source size to the target size with alpha blending to complement the transition, I'll look into, but it doesn't sound simple to implement and it will probably be CPU intensive.
One effect or the other, two at one will surely be too slow. I think alpha alone would be the simplest to implement, as you just have to replace contents and run the animation.
And I was suggesting the configuration option to disable animations especially for people that are annoyed by them and also for people with old computers :)
Sounds reasonable. I think this could be implemented by setting jQuery.fx.off to true.
I've improved the animation to only slide once during the replacement of the content and uploaded a new demo video [1].
Please let me know what you think.
[1]: http://www.placella.com/pma/improved_routine_animations.ogv