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

Added image preview for HomeKey Card Finish in Misc configuration page on HomeKey Card Finish click #114

Merged
merged 4 commits into from
Aug 23, 2024

Conversation

adukale
Copy link

@adukale adukale commented Aug 19, 2024

This pull request adds image preview for HomeKey Card Finish in Misc configuration page on HomeKey Card Finish click.

gif showing the UI:
hk-finish-preview

If you accept this commit then we can remove the images from littlefs and add raw links of images in code, which will free up the space.
Let me know.

@rednblkx
Copy link
Owner

Hi! Appreciate the PR 😀

Got some thoughts on this:

  • The images are taking up too much space and i do not intend to embed external links into the webui.

  • All images are too high-res for this purpose and hence have a lot of redundant data, resizing them will help greatly, as an example, resizing hk-finish-0.webp to 256x162 is plenty enough and brings it down from 7.0 KiB to 710 B

hk-finish-0_resized

Right now am working on reorganizing and refreshing the design of the webui, once done i'll rebase the PR and merge it in along with the resized the images

@adukale
Copy link
Author

adukale commented Aug 21, 2024

Great. Let me know if you need any help. Happy to help.

@rednblkx rednblkx merged commit d48287d into rednblkx:dev Aug 23, 2024
2 checks passed
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