Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
ensure that we test every value for the length datatype
  • Loading branch information
RobinMalfait committed Dec 6, 2021
commit e7bd118078e34efa7bf3d53d1165e8a1b7c0da09
12 changes: 7 additions & 5 deletions src/util/dataTypes.js
Original file line number Diff line number Diff line change
Expand Up @@ -80,11 +80,13 @@ let lengthUnits = [
]
let lengthUnitsPattern = `(?:${lengthUnits.join('|')})`
export function length(value) {
return (
value === '0' ||
new RegExp(`${lengthUnitsPattern}$`).test(value) ||
cssFunctions.some((fn) => new RegExp(`^${fn}\\(.+?${lengthUnitsPattern}`).test(value))
)
return value.split(UNDERSCORE).every((part) => {
return (
part === '0' ||
new RegExp(`${lengthUnitsPattern}$`).test(part) ||
cssFunctions.some((fn) => new RegExp(`^${fn}\\(.+?${lengthUnitsPattern}`).test(part))
)
})
}

let lineWidths = new Set(['thin', 'medium', 'thick'])
Expand Down
3 changes: 3 additions & 0 deletions tests/arbitrary-values.test.css
Original file line number Diff line number Diff line change
Expand Up @@ -666,6 +666,9 @@
.bg-\[length\:var\(--value\)\] {
background-size: var(--value);
}
.bg-\[center_top_1rem\] {
background-position: center top 1rem;
}
.bg-\[position\:200px_100px\] {
background-position: 200px 100px;
}
Expand Down
1 change: 1 addition & 0 deletions tests/arbitrary-values.test.html
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,7 @@
<div class="bg-[length:200px_100px]"></div>
<div class="bg-[length:var(--value)]"></div>

<div class="bg-[center_top_1rem]"></div>
<div class="bg-[position:200px_100px]"></div>
<div class="bg-[position:var(--value)]"></div>

Expand Down