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

Remove Dynamic <script> tags from widget definitions, and make the whole thing work with inline fields. #23

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

martyzz1
Copy link

@martyzz1 martyzz1 commented May 11, 2018

I couldn't test the AutoCompleteMultipleWidget but I've converted it as best as I can tell. But any help testing it would be appreciated...

I've tested the AutoCompleteWidget, and it works

I'll push an update to the docs, there's no need to install the scripts by overridding admin templates now

@martyzz1 martyzz1 changed the title Fix broken display strings in results not following unicode or str Remove Dynamic <script> tages from widget definitions, and make the whole thing work with inline fields. May 11, 2018
@martyzz1 martyzz1 changed the title Remove Dynamic <script> tages from widget definitions, and make the whole thing work with inline fields. Remove Dynamic <script> tags from widget definitions, and make the whole thing work with inline fields. May 11, 2018
@hedleyroos
Copy link
Contributor

Thanks! I'll test it locally as soon as possible and do the merge.

@martyzz1
Copy link
Author

martyzz1 commented May 15, 2018 via email

@martyzz1
Copy link
Author

This versions now seems to work happily with Whitenoise...
If those images are deprecated from jquiery-ui then we'd need to remove them from the css as well as the repo...

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