Account created on 19 December 2009, over 14 years ago
#

Merge Requests

More

Recent comments

πŸ‡ΊπŸ‡ΈUnited States sethhill

This change looks good. Will merge shortly.

πŸ‡ΊπŸ‡ΈUnited States sethhill

sethhill β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States sethhill

MR adds a check for creation permission of a certain paragraph type in order to control visibility of the duplication controls.

πŸ‡ΊπŸ‡ΈUnited States sethhill

This ended up fitting in with tests that I was writing for https://www.drupal.org/project/layout_paragraphs/issues/3440313 ✨ Allow View Mode Selection Inside Layout Paragraphs Widget Fixed , and I added this functionality into MR!167 on that ticket.

πŸ‡ΊπŸ‡ΈUnited States sethhill

Rerolled the patch in #61 against 10.3.x

πŸ‡ΊπŸ‡ΈUnited States sethhill

This appears to be a duplicate of https://www.drupal.org/project/layout_paragraphs/issues/3265794 πŸ› Modal form actions broken if subform has validation errors or certain AJAX callbacks Fixed . Fix is already merged into 2.0.x-dev.

πŸ‡ΊπŸ‡ΈUnited States sethhill

sethhill β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States sethhill

@Nelo_Drup, can you please provide steps to recreate? In particular, what versions of Layout Paragraphs and Mercury Editor are causing this behavior for you?

πŸ‡ΊπŸ‡ΈUnited States sethhill

I added to the patch that @swilmes posted, displaying the view mode and preview mode by name. I also added the view_mode to the config schema and ensured that phpunit tests are still successful. This looks good from my perspective.

πŸ‡ΊπŸ‡ΈUnited States sethhill

sethhill β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States sethhill

This looks good to me. I'm able to see the _layoutParagraphsLayout and _layoutParagraphsBuilder properties when viewing paragraphs within a LP layout or builder, respectively.

πŸ‡ΊπŸ‡ΈUnited States sethhill

Noting that this solution is the same as what's suggested at https://www.drupal.org/project/layout_paragraphs/issues/3397222 πŸ› Widget is hidden if untranslatable fields are hidden Needs review , where I also updated the builder.js to pass FunctionalJavascript tests on the latest 2.0.x code. Once that is merged this should be updated as fixed as well.

πŸ‡ΊπŸ‡ΈUnited States sethhill

@justin2pin This looked good, although I discovered that FunctionalJavascript tests were failing with the latest 2.0.x code.

πŸ‡ΊπŸ‡ΈUnited States sethhill

This looks good, setting to RBTC.

πŸ‡ΊπŸ‡ΈUnited States sethhill

sethhill β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States sethhill

sethhill β†’ changed the visibility of the branch 3265794-modal-actions-refactor to hidden.

πŸ‡ΊπŸ‡ΈUnited States sethhill

sethhill β†’ made their first commit to this issue’s fork.

πŸ‡ΊπŸ‡ΈUnited States sethhill

The above approach by @nicross-com of updating the dialog buttons works as expected. In MR156 I refactored that approach to avoid setting the global variable to track the dialog, and addressed some ESLint errors that were reported in the builder.

Production build 0.69.0 2024