Skip to content

refactor: cleanup Takumi glue code#490

Merged
harlan-zw merged 4 commits intonuxt-modules:mainfrom
kane50613:main
Feb 28, 2026
Merged

refactor: cleanup Takumi glue code#490
harlan-zw merged 4 commits intonuxt-modules:mainfrom
kane50613:main

Conversation

@kane50613
Copy link
Copy Markdown
Contributor

@kane50613 kane50613 commented Feb 28, 2026

🔗 Linked issue

npmx-dev/npmx.dev#1654
#435

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Takumi has built-in

  • color-mix()
  • gradient with color space
  • container units
  • calc()

#435 Uint8Array not owned

@harlan-zw harlan-zw merged commit d1ccdde into nuxt-modules:main Feb 28, 2026
1 check passed
@harlan-zw
Copy link
Copy Markdown
Collaborator

Thanks for the help, much appreicated.

@harlan-zw
Copy link
Copy Markdown
Collaborator

Looks like a small regression in here if we have an SVG like so:

  <img
          src="/logo-icon.svg"
          width="60"
          height="60"
          alt="npmx logo"
        />

It won't respect the width/height attributes and instead uses the width/height on the SVG. Have to add explicit width / heihgt with style to fix.

@kane50613
Copy link
Copy Markdown
Contributor Author

Looks like a small regression in here if we have an SVG like so:

  <img
          src="/logo-icon.svg"
          width="60"
          height="60"
          alt="npmx logo"
        />

It won't respect the width/height attributes and instead uses the width/height on the SVG. Have to add explicit width / heihgt with style to fix.

May I know what version did you used?

@kane50613
Copy link
Copy Markdown
Contributor Author

@harlan-zw fixed in 0.70.2!

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.

2 participants