Skip to content
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

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

Befkadu1
Copy link
Contributor

@Befkadu1 Befkadu1 commented Jan 8, 2025

Review:

  • Commits are atomic
  • Commits have the correct type for the changes made
  • Commits with breaking changes are marked as such

Browsers tested:

(Check any that applies, it's ok to leave boxes unchecked if testing something didn't seem relevant.)

Windows:

  • Chrome
  • Edge
  • Firefox

Linux:

  • Chrome
  • Firefox

macOS:

  • Chrome
  • Firefox
  • Safari

Mobile:

  • Chrome on Android
  • iOS

Copy link

github-actions bot commented Jan 8, 2025

Documentation has been published to https://lundalogik.github.io/lime-elements/versions/PR-3392/

@Befkadu1 Befkadu1 force-pushed the show-pics-list-item branch from 6996f0c to a70b05c Compare January 8, 2025 15:48
@Befkadu1 Befkadu1 changed the title wip add support for displaying pictures in the list items Jan 8, 2025
@Befkadu1 Befkadu1 marked this pull request as ready for review January 8, 2025 15:50
@Befkadu1 Befkadu1 requested a review from a team as a code owner January 8, 2025 15:50
@Befkadu1 Befkadu1 force-pushed the show-pics-list-item branch from a70b05c to d7f4aca Compare January 9, 2025 08:45
@Befkadu1 Befkadu1 enabled auto-merge (rebase) January 9, 2025 08:45
@@ -2294,6 +2294,7 @@ export interface ListItem<T = any> {
icon?: string | Icon;
// @deprecated
iconColor?: string;
image?: Image_2;
Copy link
Contributor

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?

Copy link
Contributor Author

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

Copy link
Contributor

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:

image image

Copy link
Contributor

@adrianschmidt adrianschmidt left a 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 👍

@Befkadu1 Befkadu1 merged commit ebae96e into main Jan 9, 2025
12 checks passed
@Befkadu1 Befkadu1 deleted the show-pics-list-item branch January 9, 2025 10:42
@lime-opensource
Copy link
Collaborator

🎉 This PR is included in version 37.79.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants