Skip to content

Conversation

@ericfennis
Copy link
Member

@ericfennis ericfennis commented Dec 4, 2025

Adjustments site

  • Color pick with "normal" value
Screenshot 2025-12-04 at 15 04 31
  • Searchbar with a clear button
image
  • Removed the Flutter package from the homepage

@ericfennis ericfennis added the 🌍 site Has to do something with the Lucide website label Dec 4, 2025
set: (val) => emit('update:modelValue', val)
})
get: () => {
if (props.modelValue === 'currentColor') {
Copy link
Member

Choose a reason for hiding this comment

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

How about?

Suggested change
if (props.modelValue === 'currentColor') {
if (!props.modelValue || props.modelValue === 'currentColor') {

Copy link
Member Author

Choose a reason for hiding this comment

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

This can be annoying if you want to clear the field. But we can make it explicit.

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Add check if value is undefined or null

@ericfennis ericfennis merged commit 3b0d158 into main Dec 12, 2025
3 checks passed
@ericfennis ericfennis deleted the small-fixes-site branch December 12, 2025 08:25
karsa-mistmere added a commit that referenced this pull request Dec 12, 2025
… bar (#3851)

* Small adjustments

* Format code

* format code

* Remove default value

* format code

* update yml file

* Format code

* Update docs/.vitepress/theme/components/base/Input.vue

Co-authored-by: Karsa <[email protected]>

* Add extra check if null or undefined

---------

Co-authored-by: Karsa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌍 site Has to do something with the Lucide website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants