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

[add] exhibit namespaces makes chameleon happy. addresses #1 #4

Merged
merged 1 commit into from
Jan 9, 2013

Conversation

simahawk
Copy link
Member

@simahawk simahawk commented Jan 9, 2013

No description provided.

avoinea added a commit that referenced this pull request Jan 9, 2013
[add] exhibit namespaces makes chameleon happy. addresses #1
@avoinea avoinea merged commit e97f6cd into collective:master Jan 9, 2013
@avoinea
Copy link
Member

avoinea commented Jan 9, 2013

This breaks the view when using without chameleon https://gist.github.com/654d0fa158da499aade2

<tal:facet define="
name python:view.gett('name', '');
label python:view.gett('label', '');">
<tal:block xmlns:ex="http://simile.mit.edu/2006/11/exhibit#">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@avoinea
Copy link
Member

avoinea commented Jan 9, 2013

I think it doesn't work with the "tal" element, it seems to work ok with div and other HTML elements. I just pushed a quick fix in eea.app.visualization and eea.exhibit. Can you please check if this is ok for chameleon?

Thank you

@simahawk
Copy link
Member Author

I confirm, it works :) Sorry for the inconvenient :P

@avoinea
Copy link
Member

avoinea commented Jan 10, 2013

:) Don't worry.

Can you please also drop a line in HISTORY.txt with your changes? Same for eea.exhibit. Thanks

@simahawk
Copy link
Member Author

done!

@simahawk simahawk deleted the fix-namespaces branch January 10, 2013 11:07
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.

2 participants