Skip to content

Conversation

@RobinMalfait
Copy link
Member

@RobinMalfait RobinMalfait commented Mar 12, 2025

This PR fixes an issue where if you use a number with a decimal in a variant then it wasn't picked up correctly.

E.g.:

<div class="2xl:flex 1.5xl:flex"></div>
            ^^^^^^^^                        Picked up
                     ^^^^^^^^^^             Not picket up

This PR fixes that behavior by applying the same rules for utilities where a . is valid if it is surrounded by numbers.

Test plan

  1. Added test to ensure this is picked up
  2. Existing tests pass
  3. Ran the extractor on a real example with the following results:
Before After
image image

They are crossed out just because it's not a default value we know in the system, but you can see that the 1. part is also extracted now.

Fixes: #17148

@RobinMalfait RobinMalfait marked this pull request as ready for review March 12, 2025 19:15
@RobinMalfait RobinMalfait requested a review from a team as a code owner March 12, 2025 19:15
Copy link
Contributor

@thecrypticace thecrypticace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm (aside from the comment)

@RobinMalfait RobinMalfait enabled auto-merge (squash) March 12, 2025 21:03
@RobinMalfait RobinMalfait merged commit cedd54f into main Mar 12, 2025
5 checks passed
@RobinMalfait RobinMalfait deleted the fix/issue-17148 branch March 12, 2025 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Version > 4.0.11 do not recognize dots in the custom screens breakpoints

4 participants