Skip to content

Divi#3068

Draft
brianhogg wants to merge 21 commits intodevfrom
divi
Draft

Divi#3068
brianhogg wants to merge 21 commits intodevfrom
divi

Conversation

@brianhogg
Copy link
Copy Markdown
Contributor

@brianhogg brianhogg commented Dec 12, 2025

Description

Initial core Divi 4 support with LifterLMS modules

yarn build in the includes/divi folder rebuilds the styling and JS files.

For todos see the post mortem. It would be good if the 'disable default lifterlms content' showed the correct toggle value after refresh, and only appeared on the Course Settings (or LLMS post types) rather than everywhere. It should be possible to restrict since there are filters for Woocommerce post types to rename things like the Excerpt to Short Description.

How has this been tested?

Manually

Screenshots

CleanShot 2025-12-12 at 14 35 45@2x CleanShot 2025-12-12 at 14 36 10@2x

Checklist:

  • This PR requires and contains at least one changelog file.
  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

… field. Will see if temp "heading" can be removed.
…han the shortcode. Otherwise divi takes over the rendering.
…ng the content correctly but doesn't show in the visual editor.
…ing the content correctly but doesn't show in the visual editor.
@ideadude ideadude moved this to Awaiting Review in Development Dec 12, 2025
@@ -0,0 +1,35 @@
// External Dependencies
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Can likely delete this

@brianhogg brianhogg added this to the 10.0 milestone Dec 15, 2025
@brianhogg brianhogg marked this pull request as draft January 8, 2026 15:00
@brianhogg brianhogg moved this from Awaiting Review to Blocked in Development Jan 12, 2026
@brianhogg brianhogg removed this from the 10.0 milestone Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Blocked

Development

Successfully merging this pull request may close these issues.

2 participants