Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix negative utility generation and detection when using a prefix #7295

Merged
merged 7 commits into from Feb 7, 2022
Merged

Fix negative utility generation and detection when using a prefix #7295

merged 7 commits into from Feb 7, 2022

Conversation

thecrypticace
Copy link
Collaborator

@thecrypticace thecrypticace commented Feb 1, 2022

We now consistently handle generation and detection of negative utilities when using prefixes.

Examples of what's fixed:

  1. A utility like -tw-top-1 now generates .-tw-top-1 instead of .tw--top-1.
  2. A utility like hover:-tw-top-1 now generates .hover\:-tw-top-1:hover instead of .tw--top-1.
  3. Applying a utility with a variant like @apply hover:-tw-top-1 now generates the expected output instead of crashing.
  4. A safelist pattern of /-tw-top-1/g now generates .-tw-top-1 instead of .tw--top-1.
  5. A safelist pattern of /-tw-top-1/g with varients now generate all of the expected variants instead of just the bare utility.

Fixes #6981
Fixes #6861

@thecrypticace
Add failing tests for negative utility detection
3e03d21 
We're not generating them properly in all cases, when using at-apply we sometimes crash, and safelisting doesn't currently work as expected.
@thecrypticace
Refactor
e70733c
@thecrypticace thecrypticace merged commit 01fbe19 into master Feb 7, 2022
10 checks passed
@thecrypticace thecrypticace deleted the fix/negative-prefix branch Feb 7, 2022
@bradlc bradlc mentioned this pull request Feb 15, 2022
Closed
dukex added a commit to dukex/tailwindcss-rails that referenced this issue Feb 15, 2022
@dukex
Update tailwindcss to v3.0.22
5ac6105 
# CHANGELOG

## [3.0.22] - 2022-02-11

### Fixed

- Temporarily move postcss to dependencies ([#7424](tailwindlabs/tailwindcss#7424))

## [3.0.21] - 2022-02-10

### Fixed

- Move prettier plugin to dev dependencies ([#7418](tailwindlabs/tailwindcss#7418))

## [3.0.20] - 2022-02-10

### Added

- Expose `context.sortClassList(classes)` ([#7412](tailwindlabs/tailwindcss#7412))

## [3.0.19] - 2022-02-07

### Fixed

- Fix preflight border color fallback ([#7288](tailwindlabs/tailwindcss#7288))
- Correctly parse shadow lengths without a leading zero ([#7289](tailwindlabs/tailwindcss#7289))
- Don't crash when scanning extremely long class candidates ([#7331](tailwindlabs/tailwindcss#7331))
- Use less hacky fix for URLs detected as custom properties ([#7275](tailwindlabs/tailwindcss#7275))
- Correctly generate negative utilities when dash is before the prefix ([#7295](tailwindlabs/tailwindcss#7295))
- Detect prefixed negative utilities in the safelist ([#7295](tailwindlabs/tailwindcss#7295))

## [3.0.18] - 2022-01-28

### Fixed

- Fix `@apply` order regression (in `addComponents`, `addUtilities`, ...) ([#7232](tailwindlabs/tailwindcss#7232))
- Quick fix for incorrect arbitrary properties when using URLs ([#7252](tailwindlabs/tailwindcss#7252))

## [3.0.17] - 2022-01-26

### Fixed

- Remove false positive warning in CLI when using the `--content` option ([#7220](tailwindlabs/tailwindcss#7220))

## [3.0.16] - 2022-01-24

### Fixed

- Ensure to transpile the PostCSS Nesting plugin (tailwindcss/nesting) ([#7080](tailwindlabs/tailwindcss#7080))
- Improve various warnings ([#7118](tailwindlabs/tailwindcss#7118))
- Fix grammatical mistake ([cca5a38](tailwindlabs/tailwindcss@cca5a38))
@dukex dukex mentioned this pull request Feb 15, 2022
Merged
dhh pushed a commit to rails/tailwindcss-rails that referenced this issue Feb 19, 2022
@dukex
Update tailwindcss to v3.0.22 (#143)
0193dd7 
# CHANGELOG

## [3.0.22] - 2022-02-11

### Fixed

- Temporarily move postcss to dependencies ([#7424](tailwindlabs/tailwindcss#7424))

## [3.0.21] - 2022-02-10

### Fixed

- Move prettier plugin to dev dependencies ([#7418](tailwindlabs/tailwindcss#7418))

## [3.0.20] - 2022-02-10

### Added

- Expose `context.sortClassList(classes)` ([#7412](tailwindlabs/tailwindcss#7412))

## [3.0.19] - 2022-02-07

### Fixed

- Fix preflight border color fallback ([#7288](tailwindlabs/tailwindcss#7288))
- Correctly parse shadow lengths without a leading zero ([#7289](tailwindlabs/tailwindcss#7289))
- Don't crash when scanning extremely long class candidates ([#7331](tailwindlabs/tailwindcss#7331))
- Use less hacky fix for URLs detected as custom properties ([#7275](tailwindlabs/tailwindcss#7275))
- Correctly generate negative utilities when dash is before the prefix ([#7295](tailwindlabs/tailwindcss#7295))
- Detect prefixed negative utilities in the safelist ([#7295](tailwindlabs/tailwindcss#7295))

## [3.0.18] - 2022-01-28

### Fixed

- Fix `@apply` order regression (in `addComponents`, `addUtilities`, ...) ([#7232](tailwindlabs/tailwindcss#7232))
- Quick fix for incorrect arbitrary properties when using URLs ([#7252](tailwindlabs/tailwindcss#7252))

## [3.0.17] - 2022-01-26

### Fixed

- Remove false positive warning in CLI when using the `--content` option ([#7220](tailwindlabs/tailwindcss#7220))

## [3.0.16] - 2022-01-24

### Fixed

- Ensure to transpile the PostCSS Nesting plugin (tailwindcss/nesting) ([#7080](tailwindlabs/tailwindcss#7080))
- Improve various warnings ([#7118](tailwindlabs/tailwindcss#7118))
- Fix grammatical mistake ([cca5a38](tailwindlabs/tailwindcss@cca5a38))
perez-jesus added a commit to perez-jesus/rails-tailwindcss-template that referenced this issue Apr 1, 2022
@perez-jesus
Update tailwindcss to v3.0.22 (#143)
b598960 
# CHANGELOG

## [3.0.22] - 2022-02-11

### Fixed

- Temporarily move postcss to dependencies ([#7424](tailwindlabs/tailwindcss#7424))

## [3.0.21] - 2022-02-10

### Fixed

- Move prettier plugin to dev dependencies ([#7418](tailwindlabs/tailwindcss#7418))

## [3.0.20] - 2022-02-10

### Added

- Expose `context.sortClassList(classes)` ([#7412](tailwindlabs/tailwindcss#7412))

## [3.0.19] - 2022-02-07

### Fixed

- Fix preflight border color fallback ([#7288](tailwindlabs/tailwindcss#7288))
- Correctly parse shadow lengths without a leading zero ([#7289](tailwindlabs/tailwindcss#7289))
- Don't crash when scanning extremely long class candidates ([#7331](tailwindlabs/tailwindcss#7331))
- Use less hacky fix for URLs detected as custom properties ([#7275](tailwindlabs/tailwindcss#7275))
- Correctly generate negative utilities when dash is before the prefix ([#7295](tailwindlabs/tailwindcss#7295))
- Detect prefixed negative utilities in the safelist ([#7295](tailwindlabs/tailwindcss#7295))

## [3.0.18] - 2022-01-28

### Fixed

- Fix `@apply` order regression (in `addComponents`, `addUtilities`, ...) ([#7232](tailwindlabs/tailwindcss#7232))
- Quick fix for incorrect arbitrary properties when using URLs ([#7252](tailwindlabs/tailwindcss#7252))

## [3.0.17] - 2022-01-26

### Fixed

- Remove false positive warning in CLI when using the `--content` option ([#7220](tailwindlabs/tailwindcss#7220))

## [3.0.16] - 2022-01-24

### Fixed

- Ensure to transpile the PostCSS Nesting plugin (tailwindcss/nesting) ([#7080](tailwindlabs/tailwindcss#7080))
- Improve various warnings ([#7118](tailwindlabs/tailwindcss#7118))
- Fix grammatical mistake ([cca5a38](tailwindlabs/tailwindcss@cca5a38))
Ror1211 added a commit to Ror1211/tailwindcss-rails that referenced this issue Apr 5, 2022
@Ror1211
Update tailwindcss to v3.0.22 (#143)
8310304 
# CHANGELOG

## [3.0.22] - 2022-02-11

### Fixed

- Temporarily move postcss to dependencies ([#7424](tailwindlabs/tailwindcss#7424))

## [3.0.21] - 2022-02-10

### Fixed

- Move prettier plugin to dev dependencies ([#7418](tailwindlabs/tailwindcss#7418))

## [3.0.20] - 2022-02-10

### Added

- Expose `context.sortClassList(classes)` ([#7412](tailwindlabs/tailwindcss#7412))

## [3.0.19] - 2022-02-07

### Fixed

- Fix preflight border color fallback ([#7288](tailwindlabs/tailwindcss#7288))
- Correctly parse shadow lengths without a leading zero ([#7289](tailwindlabs/tailwindcss#7289))
- Don't crash when scanning extremely long class candidates ([#7331](tailwindlabs/tailwindcss#7331))
- Use less hacky fix for URLs detected as custom properties ([#7275](tailwindlabs/tailwindcss#7275))
- Correctly generate negative utilities when dash is before the prefix ([#7295](tailwindlabs/tailwindcss#7295))
- Detect prefixed negative utilities in the safelist ([#7295](tailwindlabs/tailwindcss#7295))

## [3.0.18] - 2022-01-28

### Fixed

- Fix `@apply` order regression (in `addComponents`, `addUtilities`, ...) ([#7232](tailwindlabs/tailwindcss#7232))
- Quick fix for incorrect arbitrary properties when using URLs ([#7252](tailwindlabs/tailwindcss#7252))

## [3.0.17] - 2022-01-26

### Fixed

- Remove false positive warning in CLI when using the `--content` option ([#7220](tailwindlabs/tailwindcss#7220))

## [3.0.16] - 2022-01-24

### Fixed

- Ensure to transpile the PostCSS Nesting plugin (tailwindcss/nesting) ([#7080](tailwindlabs/tailwindcss#7080))
- Improve various warnings ([#7118](tailwindlabs/tailwindcss#7118))
- Fix grammatical mistake ([cca5a38](tailwindlabs/tailwindcss@cca5a38))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@thecrypticace