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

Entity picture option to chip entity card #268

Merged
merged 2 commits into from
Apr 24, 2022

Conversation

skrynklarn
Copy link
Contributor

Option to show the entity picture instead of icon in the chip entity card.

Increased image size to 24px instead of 1.5em that icons was. 1.5em image seemed to be very small.

Feel free to review and comment it.

@skrynklarn
Copy link
Contributor Author

Loved this project and wanted to contribute. Don't want to be rude and understand people have other things to do. But 12 days now and no response from anyone. Lot of other PR that is later that have more activity. Seems that my help is not welcome or I'm doing something wrong? Have other PR with no response either.

@piitaya
Copy link
Owner

piitaya commented Apr 21, 2022

Sorry for the late reply. I think it's a good idea. Maybe we can use a size in em instead of px to support dynamic size when changing the chip size with theme ?

@piitaya piitaya added the enhancement New feature or request label Apr 22, 2022
@chertvl
Copy link

chertvl commented Apr 22, 2022

this is what i was asking about.
thanks
#146

@piitaya
Copy link
Owner

piitaya commented Apr 24, 2022

I merge the PR and will add entity_picture to entity card too.

@piitaya piitaya merged commit aff7157 into piitaya:main Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants