Account created on 22 December 2015, over 9 years ago
#

Merge Requests

More

Recent comments

🇫🇷France mably

Sill being hit by the problem, so it looks like it's not really fixed.

Stage File Proxy is fetching .jpg styled files instead of .jpg.webp...

Any solution?

🇫🇷France mably

Sill being hit by the problem, so it looks like it's not really fixed.

Stage File Proxy is fetching .jpg styled files instead of .jpg.webp...

Any solution?

🇫🇷France mably

Yep, the MR must be slightly updated to work without jQuery but it shouldn't be too difficult.

🇫🇷France mably

I changed the MR's target to 2.0.x. Looks like a rebase is still needed though.

🇫🇷France mably

This issue's MR needs to be rebased against 2.x. Thanks.

🇫🇷France mably

Hi @lincoln-batsirayi, could you create a new MR for the 2.0.x branch? That would be great.

🇫🇷France mably

Includes fixes for the two related issues.

🇫🇷France mably

Fix included in the following issue: 📌 Switch to Vanilla JS Active .

Any feedback will be greatly appreciated.

🇫🇷France mably

Fix included in the following issue: 📌 Switch to Vanilla JS Active .

Any feedback will be greatly appreciated.

🇫🇷France mably

Can't we just use some default value when the new configuration is missing?

🇫🇷France mably

Shouldn't the rendering code also be fixed to avoid rendering raw content? 🤔

🇫🇷France mably

Working with the following exception pattern :-/\:

🇫🇷France mably

The "/" probably requires some specific escaping. Will have a look at it.

🇫🇷France mably

The "boundary exceptions" field will now be displayed only when "Match full-word" is activated:

🇫🇷France mably

Match full-word is required in fact.

We should make that clearer in the configuration form.

Could you give it another try with "match full-word" activated?

🇫🇷France mably

@sirclickalot feel free to give a try to this issue's MR.

Still wondering what kind of character, besides the hyphen possibly, we could remove from the list of word separators.

🇫🇷France mably

Ok, we are currently separating words in text using the following regexp when "full-word mode" is activated : "/\b(bit)\b/"

You want to be able to replace it with something like: "/(?<!\w|-)bit(?!\w|-)/" that removes hyphen from word separators.

Does I understand it correctly?

🇫🇷France mably

That's not a Drupal 11 feature.

Do not reopen this issue but rather create another issue if you have a specific documentation request to make.

🇫🇷France mably

4.x branch created, MR's target changed to 4.x.

@albert-volkman could you fix the Gitlab CI warnings so I can merge your MR?

🇫🇷France mably

It's a Drupal standard feature that has been added to the 3.x version of the module.

🇫🇷France mably

The cache busting parameter should be added automatically.

As you can see there: https://www.bordeaux.fr

🇫🇷France mably

Have you tried removing those ?v=blJXMRWeAF parameters from your icons urls?

They should not be there.

🇫🇷France mably

A repeatable scenario allowing to reproduce the bug on a fresh Drupal 11 instance for example.

🇫🇷France mably

More than one thousand sites are using the 3.x version without any problem.

You must provide more information to enable further investigation.

🇫🇷France mably

Thanks a lot @albert-volkman, that's a really nice upgrade!

Should we create a new 4.x development branch for this update?

🇫🇷France mably

Pushed a few fixes to MR 146. Could anyone review it, please? Thanks.

🇫🇷France mably

Added the links on the module Front Page.

I'm not sure those modules are really similar in fact.

Our module is not trying to render anything, letting that to the site's theme, but just add some extra data for improved accessibility.

🇫🇷France mably

We really need this feature, can someone review it, please?

We'll be happy to proactively answer any questions.

🇫🇷France mably

FYI it has been successfully deployed on a high traffic website (bordeaux.fr) for several weeks.

Production build 0.71.5 2024