-
-
Notifications
You must be signed in to change notification settings - Fork 870
Create a section for animations/liquid fire #448
Comments
@ef4, any update on this? If it's not going to make it for 2.0, that's okay, just want to get the status. |
@michaelrkn I can probably help draft something basic on this that we can flesh out in the future if that'd be helpful ... |
The idea is to port over http://ef4.github.io/liquid-fire/ into the Guides. @ef4, anything @acorncom can do to help out? |
Yes. The liquid-fire docs are an Ember app that's built directly from the liquid-fire repo, and I want to keep it that way, because it lets us test every change to liquid-fire against possible breakage in the docs. So the goal is to integrate the existing Ember app into the guides.emberjs.com domain. There are several possible ways to do this, so somebody just need to experiment and come up with something that works nicely. One idea is:
|
The above is all just the plumbing that's required. There is also editorial work to do too, to change the flow of the docs themselves to be more of a guide. That's where I'm probably needed. |
@ef4 Sounds good on the above. Question re: styling the liquid-fire docs app. From what I've seen, it sounds like a redesign for the Ember site is in the works in the long-run. Do you want the liquid-fire guides to use it's own copy of the Ember CSS or should I reference the guides CSS instead? Seems like a bit of a maintenance headache either way ... |
If you use the guides' CSS then it should get updated when we implement the redesign, no? |
Yes, it's just likely that somebody changes the guides CSS without noticing But I think that's the best option anyway.
|
Sounds good. I'll aim to use guides CSS for LF then. |
A few additional thoughts for us to bat around / work through as we think about this:
Would we want to embed the LiquidFire test app as an app inside of the Guides (as described here: http://guides.emberjs.com/v2.0.0/configuring-ember/embedding-applications/) ? Seems like it could cause all sorts of problems (and might not allow enough control for the LiquidFire app to run as desired) ... But doing so would allow us to create "pages" that correspond to the main LiquidFire "areas" and pre-render the Animation nav bar links for all other pages (instead of having them inaccessible until you click "Animations"). Thoughts? Am I suggesting all sorts of future pain? |
Tangential note - it would be awesome to get this issue resolved, but it is pretty low-priority compared to these. |
@michaelrkn thanks, will do 👍 Should we change the milestone on this then? I started pitching in here to help because looked like it was desired soon. Are we using the "help wanted" label for higher priority items? |
Good call. Removed the milestone. In general, the things we want help with are labelled "help wanted" :) Otherwise they are likely being worked on by somebody else or low-priority. |
@ef4 is doing this.
The text was updated successfully, but these errors were encountered: