Open hi-rai opened 1 week ago
@hi-rai is attempting to deploy a commit to the Themesberg Team on Vercel.
A member of the Team first needs to authorize it.
The changes in this pull request focus on the Button.svelte
component, specifically enhancing event handling by adding on:mouseup
and on:mousedown
event listeners for improved interaction. The use of the tag
property for rendering remains unchanged, and the overall structure, including the conditional rendering logic and type definitions, is preserved without modifications to exported properties.
File | Change Summary |
---|---|
src/lib/buttons/Button.svelte | Updated event listeners to include on:mouseup and on:mousedown ; minor formatting adjustment to CommonProps . |
Objective | Addressed | Explanation |
---|---|---|
Expose mousedown and mouseup Events in Button Component (#1490) | ✅ |
🐇 In the meadow where buttons play,
New events join the dance today!
Mousedown, mouseup, a joyful cheer,
For every click, we bring you near!
Press-and-hold, let actions unfold,
In our button world, adventures bold! 🌼
[!WARNING] There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure.
🔧 eslint
> If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.src/lib/buttons/Button.svelte
Oops! Something went wrong! :( ESLint: 9.15.0 TypeError: Error while loading rule '@typescript-eslint/no-unused-expressions': Cannot read properties of undefined (reading 'allowShortCircuit') Occurred while linting /src/lib/buttons/Button.svelte at Object.create (/node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/rules/no-unused-expressions.js:75:13) at create (/node_modules/.pnpm/@typescript-eslint+eslint-plugin@8.11.0_@typescript-eslint+parser@8.11.0_eslint@9.15.0_typesc_wgk45j5k3zhcwytjqegpxwrd2m/node_modules/@typescript-eslint/eslint-plugin/dist/rules/no-unused-expressions.js:28:32) at Object.create (/node_modules/.pnpm/@typescript-eslint+utils@8.11.0_eslint@9.15.0_typescript@5.6.3/node_modules/@typescript-eslint/utils/dist/eslint-utils/RuleCreator.js:31:20) at createRuleListeners (/node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/linter/linter.js:944:21) at /node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/linter/linter.js:1082:84 at Array.forEach () at runRules (/node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/linter/linter.js:1013:34) at #flatVerifyWithoutProcessors (/node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/linter/linter.js:1911:31) at Linter._verifyWithFlatConfigArrayAndWithoutProcessors (/node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/linter/linter.js:1993:49) at /node_modules/.pnpm/eslint@9.15.0/node_modules/eslint/lib/linter/linter.js:1609:29
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Closes #1490
📑 Description
Add mousedown and mouseup events to Button component
Status
✅ Checks
ℹ Additional Information
Summary by CodeRabbit
New Features
Bug Fixes
on:mouseup
andon:mousedown
event listeners.