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

Using html ajax request instead of json to update tree #8

Open
tdesvenain opened this issue Jan 25, 2013 · 2 comments
Open

Using html ajax request instead of json to update tree #8

tdesvenain opened this issue Jan 25, 2013 · 2 comments

Comments

@tdesvenain
Copy link

Now collective.navigationtoggle can't be adapted to specific portlet navigation customizations because of the pure javascript tree update implementation.

I suggest that the tree is updated by a html view (the response is generated by a view with html template)

I can do the implementation as I need this on a project.

@keul
Copy link
Contributor

keul commented Jan 25, 2013

To be honest the original idea behind this package is exactly this: try to copy the navigation format from the client side HTML, not calling the server.
Obviously this implementation can't be fefect.

However an (optional) alternative implementation can be a good idea (like: looking for an adapter/view, and use it if found).

If you want to provide this feature: you are welcome!

@tdesvenain
Copy link
Author

Ok i'll make a pull request, but i'll need to change the javascript in order to get html, not json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants