Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Abstract Admin Transitions #3584

Closed
rwjblue opened this issue Aug 3, 2014 · 9 comments
Closed

Abstract Admin Transitions #3584

rwjblue opened this issue Aug 3, 2014 · 9 comments
Labels
affects:admin Anything relating to Ghost Admin

Comments

@rwjblue
Copy link
Contributor

rwjblue commented Aug 3, 2014

I believe that we can significantly improve our our animations by abstracting them a bit (and avoiding jQuery.animate).

Take a look at this demo of liquid-fire a relatively new ember-cli-addon that makes this trivial with Ember CLI (and I'm sure we could leverage this without moving to Ember CLI first). Liquid Fire uses Velocity.js (which replaces jQuery.animate with a MUCH faster implementation).

IMHO, with a more abstract setup we can handle things (like the bug fixed in #3535) quite a bit better.

@ErisDS ErisDS added this to the 0.5.x Feature Release milestone Aug 3, 2014
@ErisDS ErisDS added the ember label Aug 3, 2014
@novaugust
Copy link
Contributor

#3237 is our issue to move to ember-cli, thought it might be nice to ref that from here as it's something of a blocker =)

@novaugust
Copy link
Contributor

Is velocity our only option here? I want to bring @PaulAdamDavis in on this and talk about whether we can just do this via classnames. We're seemingly doing so for the new PSM menu that's coming up, right?

@PaulAdamDavis
Copy link
Member

My first thought is to keep animations in CSS, but I need to have a play around with Liquid Fire/Velocity before I say much more :)

@novaugust
Copy link
Contributor

That's what I was asking -- if it's CSS-able at all. If it is, I personally am not interested in bringing in a dependency just yet

@rwjblue
Copy link
Contributor Author

rwjblue commented Sep 6, 2014

My point is that we should abstract the animations so that they are not done adhoc (implemented individually in each place they are needed). We should move to common components that bring the functionality we want.

How these components actually make the transitions happen (be it via assigning CSS classes or using an external tool) is less of an issue to me.

The beauty of what I am suggesting is that we could completely change the animation layer by changing a handful of these new components and everything "just works".

@novaugust
Copy link
Contributor

Ahhh i'm with you. Hence 'abstract admin transitions' :p maybe i should
reopen my css issue then..
On Sep 6, 2014 3:56 PM, "Robert Jackson" [email protected] wrote:

My point is that we should abstract the animations so that they are not
done adhoc (implemented individually in each place they are needed). We
should move to common components that bring the functionality we want.

How these components actually make the transitions happen (be it via
assigning CSS classes or using an external tool) is less of an issue to me.

The beauty of what I am suggesting is that we could completely change the
animation layer by changing a handful of these new components and
everything "just works".


Reply to this email directly or view it on GitHub
#3584 (comment).

@ErisDS
Copy link
Member

ErisDS commented Oct 9, 2015

Is this still an issue post-zelda?

@kevinansfield
Copy link
Member

There's nothing that's being animated using jquery now as far as I'm aware. If we run into deeper animation requirements (maybe once settings panes are route-driven?) then liquid-fire is a good solution.

@ErisDS
Copy link
Member

ErisDS commented Oct 9, 2015

Cool, then I'm gonna close it for now.

@ErisDS ErisDS closed this as completed Oct 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Development

No branches or pull requests

5 participants