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

plone.autoform.directives and more generic javascript #228

Merged
merged 7 commits into from May 31, 2016
Merged

Conversation

thet
Copy link
Member

@thet thet commented May 25, 2016

  • Make use of more generic selectors in JavaScript, so that JavaScript works also for derived classes.
  • Configure custom css classes for all event behavior fields.
    This makes it easier to use same selectors also for derived behaviors.
  • Use plone.autoform.directives for manipulating field widgets instead of overriding the default Fieldwidget adapters.

else:
# Plone 4
from plone.app.widgets.dx import DatetimeWidget
from plone.app.widgets.dx import DatetimeFieldWidget
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

if you skip Plone 4 support this can be removed as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

True - But removing Plone 4 support is more than that. I'd like to make a separate PR for that.

Copy link
Sponsor Member

Choose a reason for hiding this comment

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

ok, one after another.

@thet
Copy link
Member Author

thet commented May 25, 2016

I'm wondering if there is a programmatic way to be used in an upgrade step to "re-cook" javascript...

@vincentfretin
Copy link
Member

portal_javascripts.cookResources()

@thet
Copy link
Member Author

thet commented May 25, 2016

@vincentfretin although it's from the plone-legacy bundle, but in Plone 5 it's different, no?

@vincentfretin
Copy link
Member

Ah, I don't know about Plone 5.

@jensens
Copy link
Sponsor Member

jensens commented May 27, 2016

what’s the state here? do we need some "cooking" or is this sufficient?

@jensens jensens merged commit b476d1a into master May 31, 2016
@jensens jensens deleted the thet-genericjs branch May 31, 2016 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants