🇧🇪Belgium @mschudders

mschudders → created an issue.

all of these are resolved in 4.X but not in 3.X

Closing for now.

Indeed, this was resolved in the latest release.

mschudders → changed the visibility of the branch 3553318-php8.4---issue to hidden.

mschudders → created an issue.

mschudders → created an issue.

https://git.drupalcode.org/project/akamai/-/merge_requests/31

MR31 is a reroll against the latest code.

mschudders → changed the visibility of the branch 3365204-use-httpclientconfig-proxy to hidden.

mschudders → made their first commit to this issue’s fork.

https://git.drupalcode.org/issue/matomo_reports-3552072/-/commit/a6e8000...

this is the commit to review

todo review merge request.

mschudders → created an issue.

Done

mschudders → created an issue.

Fixed, thanks.

mschudders → created an issue.

Fixed, thanks.

mschudders → created an issue.

mschudders → created an issue.

I can confirm that I have the same issue when setting up a new drupal project with drupal CMS.
steps:

 cd drupalcms_displaybuilder
 ddev config --project-type=drupal11 --docroot=web
 ddev start
 ddev composer create-project drupal/cms
ddev launch
ddev composer require 'drupal/display_builder:1.0.x-dev@dev' 'drupal/ui_skins:^1.1@alpha'
ddev drush en display_builder -y

Then when you enable the default profile on for example "Blog" and when you go to "display builder" you will see this message.

I have Version: 2.0.8 of ui_patterns in which it the fix isn't yet merged as far as I can see:
https://www.drupal.org/project/ui_patterns/issues/3540536 🐛 [2.0.7] Fix callback name fallback Active
https://git.drupalcode.org/project/ui_patterns/-/commits/2.0.x

@pdureau can you merge it?

Thanks

mschudders → created an issue.

Strange I didn't see that in my browser the aria-label will check why it isn't there.

mschudders → created an issue.

Dear @mably,

Well that would work however. I strongly think and suggest that the module should be accessible (a11y) by default. As Drupal Core also is.

The other party that you mentioned in the other ticket did they do a manual accessibility test or an automated?

May I ask what they said about the change regarding the remove filter (X right now-> changing to Remove filter ABC X)

Thanks

Dear mably;

It seems (didn't check it thoroughly) most items have already been done?

Could you elaborate and what still needs to be done?

Thanks

Dear,

I don't understand that when you access(or read or get a list of links of the page) the links, a link with just "X" would be considered accessible and even user friendly.

A hook to alter would be sufficient in my case then yes.

What path are we going to follow,
1/ A Alter hook
or
2/ Having it as a configurable option.

1/ Would suit my needs.
2/ Would still need tweaking then afterwards.

Kind regards,

mably → credited mschudders → .

Please see merge request
> Needs review

mschudders → created an issue.

Needs to be review if OK.
Merge request provided.

mschudders → created an issue.

mschudders → created an issue.

added patch for older version 3.1.X:
https://git.drupalcode.org/issue/file_mdm-3319733/-/commit/7a33f4f4bb71b...
Branch: 3319733-out-of-memory-error-when-editing-node-too-many-large-fields Compare

Will test if the bumping of fileye fixes the issue.

mschudders → made their first commit to this issue’s fork.

closing.

Fixed.

added to latest DEV.

dropped 8 & 9.

Added 11

added & fixed on latest DEV.

mschudders → created an issue.

Merged.

Thanks.

added to latest DEV.

thx, merged.

not going to do anymore, was for 1.0.x

closing

done.

applied patch + dropped D8 +D9 support.

'ill pick it up next week.

doesn't apply for 2.0.x
This has been removed.

Only applies to 1.0.x

Closing.

Merged?

> Fixed.

created 2.0.x
Added new TAG and thus a new release.

reflected on project page.

closing

closing

done fixed
created new release for V2 as well.

options:
hook/event/alter > So the module who puts the values there should "hook in" and translate value to label?

mschudders → created an issue.

merged!

Thanks.

pushed on DEV.

is working well, used for several months in PRD.

fixed

closing

if you could resolve the last conflicts so I can merge the branch that would be great.

Thanks

merged looks good

thanks

If this is an option that can be toggled (and this not by default) I can agree on this.

But right now I don't think this is good to set by default.

pushed to DEV.

Thanks.

mschudders → created an issue.

kriboogh → credited mschudders → .

up to date with latest changes now.

Putting in Needs review.

mschudders → made their first commit to this issue’s fork.

Thanks!

mschudders → created an issue.

added to DEV.

Fixed.

mschudders → created an issue.

Merged!

Thanks

Fixed

Great thanks!

mschudders → created an issue.

Merged.

shouldn't be the responsability of this contrib

todo check config.

mschudders → created an issue.

merged.

Is indeed a nice request. Maybe if we have some time in the future we could implement it.

closing fixed

added to DEV.

Mschudders → created an issue.

closing and marking as fixed

I've added this to the DEV branch.

please provide MR.

thanks