Skip to content

Commit 0f033c1

Browse files
committed
fix: 移除 uni-app x 的 space 布局支持
1 parent 98841ac commit 0f033c1

File tree

10 files changed

+88
-292
lines changed

10 files changed

+88
-292
lines changed

packages/postcss/test/uni-app-x.test.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,12 @@ describe('uni-app-x', () => {
111111
.gap-4 {
112112
gap: 1rem;
113113
}
114+
.gap-x-3 {
115+
column-gap: 0.75rem;
116+
}
117+
.gap-y-2 {
118+
row-gap: 0.5rem;
119+
}
114120
.min-h-screen {
115121
min-height: 100vh;
116122
}
@@ -134,6 +140,8 @@ describe('uni-app-x', () => {
134140
expect(result.css).not.toContain('display: grid')
135141
expect(result.css).not.toContain('grid-template-columns')
136142
expect(result.css).not.toContain('gap: 1rem')
143+
expect(result.css).not.toContain('column-gap: 0.75rem')
144+
expect(result.css).not.toContain('row-gap: 0.5rem')
137145
expect(result.css).not.toContain('min-height: 100vh')
138146
expect(result.css).toContain('.flex')
139147
expect(result.css).toContain('display: flex')
@@ -146,6 +154,8 @@ describe('uni-app-x', () => {
146154
expect.stringContaining('grid'),
147155
expect.stringContaining('grid-cols-2'),
148156
expect.stringContaining('gap-4'),
157+
expect.stringContaining('gap-x-3'),
158+
expect.stringContaining('gap-y-2'),
149159
expect.stringContaining('min-h-screen'),
150160
]))
151161
expect(warningTexts.every(item => item.includes('/src/App.uvue'))).toBe(true)

packages/weapp-tailwindcss/src/uni-app-x/space-style.ts

Lines changed: 0 additions & 237 deletions
This file was deleted.

packages/weapp-tailwindcss/src/uni-app-x/transform.ts

Lines changed: 4 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import type { AttributeNode, DirectiveNode, ElementNode, ParentNode } from '@vue/compiler-dom'
1+
import type { AttributeNode, DirectiveNode, ParentNode } from '@vue/compiler-dom'
22
import type { TransformResult } from 'vite'
33
import type { CreateJsHandlerOptions, ICustomAttributesEntities, JsHandler } from '@/types'
44
import { NodeTypes } from '@vue/compiler-dom'
@@ -7,7 +7,6 @@ import MagicString from 'magic-string'
77
import { generateCode, replaceWxml } from '@/wxml'
88
import { createAttributeMatcher } from '@/wxml/custom-attributes'
99
import { shouldEnableComponentLocalStyle, UniAppXComponentLocalStyleCollector } from './component-local-style'
10-
import { UniAppXSpaceStyleCollector } from './space-style'
1110

1211
function traverse(node: ParentNode, visitor: (node: ParentNode) => void): void {
1312
visitor(node)
@@ -128,11 +127,6 @@ const defaultCreateJsHandlerOptions: CreateJsHandlerOptions = {
128127
}
129128
const UVUE_NVUE_RE = /\.(?:uvue|nvue)(?:\?.*)?$/
130129

131-
function insertClassAttribute(ms: MagicString, node: ElementNode, className: string) {
132-
const insertOffset = node.loc.start.offset + 1 + node.tag.length
133-
ms.appendLeft(insertOffset, ` class="${className}"`)
134-
}
135-
136130
export function transformUVue(
137131
code: string,
138132
id: string,
@@ -147,18 +141,11 @@ export function transformUVue(
147141
const matchCustomAttribute = createAttributeMatcher(customAttributesEntities)
148142
const ms = new MagicString(code)
149143
const { descriptor, errors } = parse(code)
150-
const spaceStyleCollector = new UniAppXSpaceStyleCollector(id)
151144
const localStyleCollector = options.enableComponentLocalStyle && shouldEnableComponentLocalStyle(id)
152145
? new UniAppXComponentLocalStyleCollector(id, runtimeSet)
153146
: undefined
154147
if (errors.length === 0) {
155148
if (descriptor.template?.ast) {
156-
traverse(descriptor.template.ast, (node) => {
157-
if (node.type === NodeTypes.ELEMENT) {
158-
spaceStyleCollector.collect(node)
159-
}
160-
})
161-
162149
traverse(descriptor.template.ast, (node) => {
163150
if (node.type !== NodeTypes.ELEMENT) {
164151
return
@@ -175,14 +162,11 @@ export function transformUVue(
175162
if (!shouldHandle) {
176163
continue
177164
}
178-
const staticClassRewrite = shouldHandleDefault
179-
? spaceStyleCollector.getStaticClassRewrite(prop)
180-
: undefined
181165
if (shouldHandleDefault && localStyleCollector) {
182-
updateStaticAttributeWithLocalStyle(ms, prop, localStyleCollector, staticClassRewrite)
166+
updateStaticAttributeWithLocalStyle(ms, prop, localStyleCollector)
183167
}
184168
else {
185-
updateStaticAttribute(ms, prop, staticClassRewrite)
169+
updateStaticAttribute(ms, prop)
186170
}
187171
if (shouldHandleDefault) {
188172
continue
@@ -204,29 +188,20 @@ export function transformUVue(
204188
if (!shouldHandle) {
205189
continue
206190
}
207-
const directiveExpressionRewrite = attrName.toLowerCase() === 'class'
208-
? spaceStyleCollector.getDirectiveExpressionRewrite(prop)
209-
: undefined
210191
if (attrName.toLowerCase() === 'class' && localStyleCollector) {
211192
updateDirectiveExpressionWithLocalStyle(
212193
ms,
213194
prop,
214195
jsHandler,
215196
localStyleCollector,
216197
runtimeSet,
217-
directiveExpressionRewrite,
218198
)
219199
}
220200
else {
221-
updateDirectiveExpression(ms, prop, jsHandler, runtimeSet, directiveExpressionRewrite)
201+
updateDirectiveExpression(ms, prop, jsHandler, runtimeSet)
222202
}
223203
}
224204
}
225-
226-
const insertedClass = spaceStyleCollector.getInsertedClass(node)
227-
if (insertedClass) {
228-
insertClassAttribute(ms, node, insertedClass)
229-
}
230205
})
231206
}
232207

@@ -252,9 +227,6 @@ export function transformUVue(
252227
if (localStyleCollector?.hasStyles()) {
253228
ms.append(`\n${localStyleCollector.toStyleBlock()}`)
254229
}
255-
if (spaceStyleCollector.hasStyles()) {
256-
ms.append(`\n${spaceStyleCollector.toStyleBlock()}`)
257-
}
258230
}
259231
return {
260232
code: ms.toString(),

0 commit comments

Comments
 (0)