Account created on 8 November 2012, over 11 years ago
  • Senior Drupal Developer at ImageXΒ  …
#

Merge Requests

More

Recent comments

πŸ‡§πŸ‡·Brazil andre.bonon

andre.bonon β†’ changed the visibility of the branch 2940605-10.0.x to hidden.

πŸ‡§πŸ‡·Brazil andre.bonon

andre.bonon β†’ changed the visibility of the branch 2940605-10.0.x to active.

πŸ‡§πŸ‡·Brazil andre.bonon

andre.bonon β†’ changed the visibility of the branch 2940605-10.0.x to active.

πŸ‡§πŸ‡·Brazil andre.bonon

andre.bonon β†’ changed the visibility of the branch 2940605-10.0.x to hidden.

πŸ‡§πŸ‡·Brazil andre.bonon

I executed the steps mentioned and applied the patch.
That works for me.

πŸ‡§πŸ‡·Brazil andre.bonon

This merge request broke ping pongs.
I rolled the version back to 9000 and tried to update again, and no new image style is created or displayed on the upgrade dashboard to be manually created.

πŸ‡§πŸ‡·Brazil andre.bonon

I tested your steps @froboy. That was very helpful.

πŸ‡§πŸ‡·Brazil andre.bonon

andre.bonon β†’ changed the visibility of the branch 1.0.x to hidden.

πŸ‡§πŸ‡·Brazil andre.bonon

Hi, I've experienced the same issue with the message being displayed to users who don't have permissions to rebuild permissions. I'm going to apply the patch and let you know if it works.
I'm attaching a patch file from the diff so we can use it with composer.

πŸ‡§πŸ‡·Brazil andre.bonon

I've added the 'layout-node-form' class to the form wrapper, so it displays the taxonomy form as the node does.
Also, I'm attaching the patch so we can use with composer.

πŸ‡§πŸ‡·Brazil andre.bonon

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

πŸ‡§πŸ‡·Brazil andre.bonon

Attached is a patch file so it can be used with Composer.

πŸ‡§πŸ‡·Brazil andre.bonon

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

πŸ‡§πŸ‡·Brazil andre.bonon

Attaching a patch to use in composer.json with the 2.1.2 version.

πŸ‡§πŸ‡·Brazil andre.bonon

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

πŸ‡§πŸ‡·Brazil andre.bonon

Attaching patch from the MR diff to use in composer.json

πŸ‡§πŸ‡·Brazil andre.bonon

Works for me.

πŸ‡§πŸ‡·Brazil andre.bonon

Attaching patch to use in composer.json

πŸ‡§πŸ‡·Brazil andre.bonon

Attaching a patch to use in composer.json

πŸ‡§πŸ‡·Brazil andre.bonon

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

πŸ‡§πŸ‡·Brazil andre.bonon

@dineshkumarbollu it works. Thanks for providing a patch for that.

πŸ‡§πŸ‡·Brazil andre.bonon

Hey, I'm attaching the code that helped me to hook it in case others need something similar.

Additionally, the code below shows how to use the Observer pattern:

  // Alter the Fullcalendar Options before rendering the calendar.
  const fullcalendarViewObserver = {
    update: (calendarOptions) => {
      // Add custom buttons.
      calendarOptions.customButtons = {
        previousMonth: {
          icon: "chevron-left",
          text: "Prev",
          click() {
            // Get some stuff done when users click on Prev button. 
          },
        },
        nextMonth: {
          icon: "chevron-right",
          text: "Next",
          click() {
            // Get some stuff done when users click on Next button. 
          },
        },
      };

      return calendarOptions;
    },
  };

  // Add a custom observer to Fullcalendar view Subject.
  Drupal.FullcalendarViewSubject.addObserver(fullcalendarViewObserver);
πŸ‡§πŸ‡·Brazil andre.bonon

Ok, I've added your suggestion and fixed the other selector as well.
Also, I unbind the event so it doesn't bind the event twice. (That was happening on the /admin/content page)

πŸ‡§πŸ‡·Brazil andre.bonon

Attached the patch from MR to use in composer.

πŸ‡§πŸ‡·Brazil andre.bonon

@aaron, we have more issues with the z-index and modals when LB is enabled for other entity types such as Blocks.
I updated the ui-dialog's z-index so as to be the same as Claro's (1260) and pushed it to the MR.
Also, I'm attaching the patch from MR to use in composer.

πŸ‡§πŸ‡·Brazil andre.bonon

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

πŸ‡§πŸ‡·Brazil andre.bonon

I reported this core bug https://www.drupal.org/project/drupal/issues/3380021 πŸ› Stable9 form-element template misses form-type-* class Fixed , so it might avoid issues with themes based on stable9.

πŸ‡§πŸ‡·Brazil andre.bonon

Hi, thanks everyone for your contribution to this issue.
I tried to fix the issue using your approach for webforms (clientside_validation + webform_clientside_validation + inline_form_errors), but the issue was still happening.
In my case, my theme extends from stable9 that doesn't add classes like "form-checkboxes", "form-radios", or "form-type-[type]" to the form elements, which are used in the webform_clientside_validation js selectors.
Stable9 adds "js-form-type-[type]" and forgets about the "form-type-[type]" as in the snippet below:

{%
  set classes = [
    'js-form-item',
    'form-item',
+        'form-type-' ~ type|clean_class, (THIS IS MISSING)
    'js-form-type-' ~ type|clean_class,
    'form-item-' ~ name|clean_class,
    'js-form-item-' ~ name|clean_class,
    title_display not in ['after', 'before'] ? 'form-no-label',
    disabled == 'disabled' ? 'form-disabled',
    errors ? 'form-item--error',
  ]
%}

I fixed the placement issue by overriding the form-element.html.twig and adding the code above. For the fieldset.html.twig, I added the following:

{%
  set classes = [
  'js-form-item',
  'form-item',
  'js-form-wrapper',
  'form-wrapper',
+  'form-' ~ element['#type'] (THIS IS MISSING)
]
πŸ‡§πŸ‡·Brazil andre.bonon

I was trying to find the same feature, but I just found out that if you add the following param "&initialDate=2023-07-28" to the URL, the JS will handle that for you.
So, if you combine the inialDate param with an exposed filter "between", I think you can achieve what you want.
i.e.: ?field_date[min]=2023-09-01&field_date[max]=2023-09-30&initialDate=2023-09-01

Note: I'm using the SmartDate module field.

πŸ‡§πŸ‡·Brazil andre.bonon

I did a test setting the "Allowed number of values" to unlimited in the field storage settings and set the cardinality limited to 4.
In the entity form, Drupal was displaying 2 fields and the 'Add another item' button before this patch was applied.
Once this patch is applied, I can see the 4 fields displayed.

πŸ‡§πŸ‡·Brazil andre.bonon

Patch with fixes as described in the issue description.

πŸ‡§πŸ‡·Brazil andre.bonon

Adding a "snapshot" patch from #8, as the MR's Diff might change.

Production build 0.69.0 2024