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

InfoSheet Enhancements #15

Closed
irvine752 opened this issue Dec 24, 2020 · 6 comments
Closed

InfoSheet Enhancements #15

irvine752 opened this issue Dec 24, 2020 · 6 comments

Comments

@irvine752
Copy link

Describe the solution you'd like
Request to add more features to the InfoSheet:

  1. InfoSheet with Lottie Animation, and text at the bottom
  2. InfoSheet with an image (url source or drawable) at the top, and text at the bottom.

*Url sources cover dynamic pictures, and the drawable covers icons + other static pictures

@maxkeppeler maxkeppeler added the enhancement New feature or request label Dec 25, 2020
@maxkeppeler
Copy link
Owner

With text at the bottom, you mean the title below the cover image / Lottie animation?

@irvine752
Copy link
Author

irvine752 commented Jan 9, 2021

yep, having it below the cover image/Lottie animation would work well. I just saw the updates in 1.1.2. The title placement you have already looks great so far, not sure if you already added the Lottie option yet.

For the icons on the image area for the Overlapping Cover Image, I would have a contrast background. I'm guessing you're setting android:src to the icon drawable, there's also android:background which should be a white circle if the icon is black. That way it's easier to see the black icons on a black or dark image.

@maxkeppeler
Copy link
Owner

Great to hear. Did you notice the 3 styles for the top view?

I did not work on Lottie animations yet.
I want to offer it as an add-on library, with which the cover image view is used to display Lottie animations and offering several more helper methods regarding that. I think it would bloat the default base library too much if it's by default included. First, I first have to figure out how I implement this though.

If you checked out the latest work for the next release, did you play around with the style(SheetStyle) available for every sheet? It displays the view as either a dialog or a bottom-sheet.

@maxkeppeler
Copy link
Owner

You are correct with the note on the missing contrast. It's possible to specify the colors for the icon buttons, so if it's known that the image is bright, dark colors can be used for the icons for a better contrast. That's correct, I'm setting the src for the icon drawable. However, I'm using the background for a ripple drawable.
When I have time, I will work on an option for the IconButtons to specify a background color, which will be seen as a round circle in the respective color behind the icon buttons!

@irvine752
Copy link
Author

Yep, I love the stuff I have seen so far. The top views should be more than enough for all the use cases. The SheetStyle is great for options, but you might have to change the name of the project though. :)

The icon background color would be a great addition. Looking forward to the next release!

@maxkeppeler
Copy link
Owner

I will, probably to just "Sheets". Ehm. Not sure when though, I would want to change the dependency names and general structure of the modules then too.

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

No branches or pull requests

2 participants