-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add support for displaying pictures in the list items #3392
Conversation
Documentation has been published to https://lundalogik.github.io/lime-elements/versions/PR-3392/ |
6996f0c
to
a70b05c
Compare
a70b05c
to
d7f4aca
Compare
@@ -2294,6 +2294,7 @@ export interface ListItem<T = any> { | |||
icon?: string | Icon; | |||
// @deprecated | |||
iconColor?: string; | |||
image?: Image_2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i know this is auto generated, but it just felt weird that it says Image_2
. Did we do something wrong, or is this correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea it is auto generated running the command npm run api:update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know. I'm not sure if I've seen that in this file before. I know I've seen it somewhere, but I'm fairly sure that's been in components.d.ts
. In this case, I find Image_2
referenced in dist/lime-elements.d.ts
, but I can't seem to find where it's actually coming from.
I've tried removing the existing dist
folder and other temporary folders, and run the build and api:update
script again, but just got the same result anyway.
It's exported as Image
anyway, so it shouldn't be a problem:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes to external interface approved 👍
🎉 This PR is included in version 37.79.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Review:
Browsers tested:
(Check any that applies, it's ok to leave boxes unchecked if testing something didn't seem relevant.)
Windows:
Linux:
macOS:
Mobile: