Skip to content

Conversation

@jpjpjp
Copy link
Contributor

@jpjpjp jpjpjp commented Nov 2, 2025

Problem

Currently, the buttons documentation does not describe the image-dark property. When viewing this page in light mode, you cannot see the scalar icon that is added to the button in the last example

Solution

With this PR, we include icon-dark as an optional property in the properties documentation and update the example to show a button with both light and dark images.

Checklist

I've gone through the following:

  • I've added an explanation why this change is needed.
  • I've added a changeset (pnpm changeset).
  • I've added tests for the regression or new feature.
  • I've updated the documentation.

Note

Adds icon-dark as an optional button prop and updates the external icon example to demonstrate light/dark icons.

  • Documentation
    • Buttons (documentation/guides/docs/components/buttons.md):
      • Add icon-dark optional prop to the Properties section with description.
      • Update "Button with External Icon" example to include both icon (light) and icon-dark (dark), adjusting example values across HTML and directive snippets.

Written by Cursor Bugbot for commit e48555a. This will update automatically on new commits. Configure here.

@changeset-bot
Copy link

changeset-bot bot commented Nov 2, 2025

⚠️ No Changeset found

Latest commit: e48555a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Member

@hanspagel hanspagel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you’re right, thanks!

can we use the following URLs for the logo?

https://cdn.scalar.com/images/logo-dark.svg
https://cdn.scalar.com/images/logo-light.svg

the other one will go away soon

@hanspagel hanspagel force-pushed the document_image_dark branch from 8291217 to e48555a Compare November 3, 2025 14:53
@jpjpjp
Copy link
Contributor Author

jpjpjp commented Nov 4, 2025

can we use the following URLs for the logo?

no problem. Finding a logo to use actually took 10 times longer than creating this PR. 🤣 glad you know a good one

@hanspagel Hmmm...when I make this change locally icon-dark seems to be pulling in the wrong logo:

Screenshot 2025-11-04 at 4 13 44 PM

When I revert to the original version I committed, it works as expected:
Screenshot 2025-11-04 at 4 16 16 PM

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