Skip to content

Commit 6e58f2c

Browse files
authoredJan 29, 2022
fix: do not transform mask-type (#673)
Fix #643
1 parent d19abe2 commit 6e58f2c

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed
 
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

3-
exports[`hast-util-to-babel-ast should correctly transform svg 1`] = `"<svg width=\\"88px\\" height=\\"88px\\" viewBox=\\"0 0 88 88\\" version={1.1} xmlns=\\"http://www.w3.org/2000/svg\\" xmlnsXlink=\\"http://www.w3.org/1999/xlink\\"><title>{\\"Dismiss\\"}</title><desc>{\\"Created with Sketch.\\"}</desc><defs /><g id=\\"Blocks\\" stroke=\\"none\\" strokeWidth={1} fill=\\"none\\" fillRule=\\"evenodd\\" strokeLinecap=\\"square\\"><g id=\\"Dismiss\\" stroke=\\"#063855\\" strokeWidth={2}><path d=\\"M51,37 L37,51\\" id=\\"Shape\\" /><path d=\\"M51,51 L37,37\\" id=\\"Shape\\" /></g></g></svg>;"`;
4-
53
exports[`hast-util-to-babel-ast should handle spaces and tab 1`] = `"<svg viewBox=\\"0 0 32 32\\" xmlns=\\"http://www.w3.org/2000/svg\\"><path d=\\"M25,5h-3V3c0-1.7-1.3-3-3-3H5C3.3,0,2,1.3,2,3v20c0,1.7,1.3,3,3,3h4v1c0,2.2,1.8,4,4,4h12c2.2,0,4-1.8,4-4V9 C29,6.8,27.2,5,25,5z M5,24c-0.6,0-1-0.5-1-1V3c0-0.6,0.4-1,1-1h14c0.5,0,1,0.4,1,1v2h-6.3H13H6.3c-0.6,0-1,0.4-1,1s0.4,1,1,1h3.2 C9.4,7.3,9.2,7.7,9.1,8C9.1,8,9,8,9,8H6.5c-0.6,0-1,0.4-1,1s0.4,1,1,1H9v3c-0.1,0-0.1,0-0.2,0H6.1c-0.6,0-1,0.4-1,1s0.4,1,1,1h2.7 c0.1,0,0.1,0,0.2,0V16c-0.1,0-0.1,0-0.2,0H6.1c-0.6,0-1,0.4-1,1c0,0.6,0.4,1,1,1h2.7c0.1,0,0.1,0,0.2,0V19c-0.1,0-0.1,0-0.2,0H6.1 c-0.6,0-1,0.4-1,1s0.4,1,1,1h2.7c0.1,0,0.1,0,0.2,0v3H5z M27,27c0,1.1-0.9,2-2,2H13c-1.1,0-2-0.9-2-2V9c0-1.1,0.9-2,2-2h0.7H25 c1.1,0,2,0.9,2,2V27z M25.2,19c0,0.6-0.4,1-1,1H13.4c-0.6,0-1-0.4-1-1s0.4-1,1-1h10.7C24.7,18,25.2,18.4,25.2,19z M25.2,22 c0,0.6-0.4,1-1,1H13.4c-0.6,0-1-0.4-1-1s0.4-1,1-1h10.7C24.7,21,25.2,21.4,25.2,22z M25.2,25c0,0.6-0.4,1-1,1H13.4c-0.6,0-1-0.4-1-1 s0.4-1,1-1h10.7C24.7,24,25.2,24.4,25.2,25z M12.3,11c0-0.6,0.4-1,1-1h7.3c0.6,0,1,0.4,1,1s-0.4,1-1,1h-7.3 C12.8,12,12.3,11.6,12.3,11z M16,13c0.6,0,1,0.4,1,1s-0.4,1-1,1h-2.5c-0.6,0-1-0.4-1-1s0.4-1,1-1H16z\\" /></svg>;"`;
4+
5+
exports[`hast-util-to-babel-ast transforms SVG 1`] = `"<svg width=\\"88px\\" height=\\"88px\\" viewBox=\\"0 0 88 88\\" version={1.1} xmlns=\\"http://www.w3.org/2000/svg\\" xmlnsXlink=\\"http://www.w3.org/1999/xlink\\"><title>{\\"Dismiss\\"}</title><desc>{\\"Created with Sketch.\\"}</desc><defs /><g id=\\"Blocks\\" stroke=\\"none\\" strokeWidth={1} fill=\\"none\\" fillRule=\\"evenodd\\" strokeLinecap=\\"square\\"><g id=\\"Dismiss\\" stroke=\\"#063855\\" strokeWidth={2}><path d=\\"M51,37 L37,51\\" id=\\"Shape\\" /><path d=\\"M51,51 L37,37\\" id=\\"Shape\\" /></g></g></svg>;"`;

‎packages/hast-util-to-babel-ast/src/index.test.ts

+9-4
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ function transform(code: string) {
1313
}
1414

1515
describe('hast-util-to-babel-ast', () => {
16-
it('should correctly transform svg', () => {
16+
it('transforms SVG', () => {
1717
const code = `
1818
<?xml version="1.0" encoding="UTF-8"?>
1919
<svg width="88px" height="88px" viewBox="0 0 88 88" version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
@@ -32,27 +32,32 @@ describe('hast-util-to-babel-ast', () => {
3232
expect(transform(code)).toMatchSnapshot()
3333
})
3434

35-
it('should correctly transform aria-x', () => {
35+
it('transforms "aria-x"', () => {
3636
const code = `<svg aria-hidden="true"></svg>`
3737
expect(transform(code)).toMatchInlineSnapshot(
3838
`"<svg aria-hidden=\\"true\\" />;"`,
3939
)
4040
})
4141

42-
it('should correctly transform aria-xxxXxx', () => {
42+
it('transforms "aria-xxxXxx"', () => {
4343
const code = `<svg aria-labelledby="foo" aria-describedat="foo" aria-describedby="foo"></svg>`
4444
expect(transform(code)).toMatchInlineSnapshot(
4545
`"<svg aria-labelledby=\\"foo\\" aria-describedat=\\"foo\\" aria-describedby=\\"foo\\" />;"`,
4646
)
4747
})
4848

49-
it('should correctly transform data-x', () => {
49+
it('transformss "data-x"', () => {
5050
const code = `<svg data-hidden="true"></svg>`
5151
expect(transform(code)).toMatchInlineSnapshot(
5252
`"<svg data-hidden=\\"true\\" />;"`,
5353
)
5454
})
5555

56+
it('preserves "mask-type"', () => {
57+
const code = `<svg><mask mask-type="alpha" /></svg>`
58+
expect(transform(code)).toBe('<svg><mask mask-type="alpha" /></svg>;')
59+
})
60+
5661
it('should handle spaces and tab', () => {
5762
const code = `<svg viewBox="0 0 32 32" xmlns="http://www.w3.org/2000/svg">
5863
<path d="M25,5h-3V3c0-1.7-1.3-3-3-3H5C3.3,0,2,1.3,2,3v20c0,1.7,1.3,3,3,3h4v1c0,2.2,1.8,4,4,4h12c2.2,0,4-1.8,4-4V9

‎packages/hast-util-to-babel-ast/src/mappings.ts

-1
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,6 @@ export const ATTRIBUTE_MAPPING = {
299299
mask: 'mask',
300300
maskcontentunits: 'maskContentUnits',
301301
maskunits: 'maskUnits',
302-
'mask-type': 'maskType',
303302
mathematical: 'mathematical',
304303
mode: 'mode',
305304
numoctaves: 'numOctaves',

1 commit comments

Comments
 (1)

vercel[bot] commented on Jan 29, 2022

@vercel[bot]
Please sign in to comment.