Mathieu Lamiot

Results 242 comments of Mathieu Lamiot

I opened a PR #416 based on this suggestion: > Implementing such functionality: if line from diff is not being reported (is missing in the report), going up line by...

After discussing with @wordpressfan, it could be a good opportunity to combine this with WP Launchpad: instead of copy/pasting files between repositories, we could have a common library for all...

@wordpressfan @CrochetFeve0251 Do you have an update on the targeted approach here? Thanks

@piotrbak Gutenberg 17.4 is not released yet (RC01 ongoing). Should this issue be delayed to Imagify 2.1.5 then?

@wordpressfan I assigned you this ticket as it seems you are working on it, based on [this Slack thread](https://wp-media.slack.com/archives/CUKB44GNN/p1701077939006009?thread_ts=1700584532.672159&cid=CUKB44GNN):

Here is an up-to-date branch with the patch from Trac, but on GH to facilitate our work. I had to adapt a few things, and also switch a behavior: Now,...

Sorry, forgot the link: https://github.com/wp-media/wordpress-develop/tree/enhancement/typed-apply-filters

Instructions to follow-up: https://wp-media.slack.com/archives/CUT7FLHF1/p1718998379118069

Thank you @CrochetFeve0251 About the PR itself on dispatcher: - I see there are dedicated methods defined for apply_filter with int, and float for example. Having to create a dedicated...