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

Added webpack with support for ES6, updated packages and more #319

Closed
wants to merge 2 commits into from
Closed

Conversation

arjendejong12
Copy link

In this pull request I've integrated webpack with babel for scripts compiling instead of compiling with only gulp-babel, mainly to fix the ResponsiveAccordeonTabs plugin, which was not working correctly. This is also more future-proof, if you want to integrate frameworks like Vue or React and use their templating systems (like I've done for a project). I've also updated packages and replaced a slew of gulp-{plugin} for their original counterpart in order to get better update support. I've added a Prettier config file for formatting consistency. The translation folder has been moved to assets and the script files have been overhauled to support ES6 syntax (and this is where you need to comment out Foundation plugins you don't use now). The gulpfile has seen many changes to incorporate the above and lastly I've added support for production/development environments in the scripts you run to make changes in this theme.

@JeremyEnglert
Copy link
Owner

This is pretty awesome! Unfortunately, I think this is too large of a change to push into the current version. However, I'd love to discuss this for JointsWP 6: #366

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

3 participants