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

V0.6 #93

Merged
merged 1 commit into from
Feb 6, 2012
Merged

V0.6 #93

merged 1 commit into from
Feb 6, 2012

Conversation

aadsm
Copy link
Collaborator

@aadsm aadsm commented Feb 4, 2012

...ent

When the content of the slot is needsDraw=true and set an element it is not being added to the draw loop because it is not part of the component tree since its element isn't in the DOM yet.

When the content is finally added to the component tree needsDraw=true doesn't have any effect anymore because it is already in that state, never making it to the draw cycle ever again.

This will be correctly addressed in master.

…he slot content

When the content of the slot is needsDraw=true and set an element it is not being added to the draw loop because it is not part of the component tree since its element isn't in the DOM yet.

When the content is finally added to the component tree needsDraw=true doesn't have any effect anymore because it is already in that state, never making it to the draw cycle ever again.

This will be correctly addressed in master.
francoisfrisch pushed a commit that referenced this pull request Feb 6, 2012
Temporary fix to slot drawing
@francoisfrisch francoisfrisch merged commit 4c8c6c4 into montagejs:v0.6 Feb 6, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants