Tibolte / ElasticDownload

We are not Gif makers, We are developers
1.65k stars 266 forks source link

Bug on #24

Closed Ashok-Varma closed 9 years ago

Ashok-Varma commented 9 years ago

i followed your example. i works fine. But if we come back to the activity. Its not showing download button by default

I mean we have two activities. we went from 1st to 2nd and in 2nd activity we have your view. First time when we open 2nd activity from 1st, it works fine. But Once we start some download (failed or downloaded) and then when i go back to 1st and come back again to second instead of showing download its showing 3rd image.

Solution:- Just add method like elasticdownloader.showDownloadButton() which shows download button.

Please also try to add methods to change the colors of download button

First time i opened

screenshot_2015-08-03-15-26-59

Fail or pass screenshot_2015-08-03-15-33-03

Went back to first activity an came back to 2nd i shows the image below instead of download button screenshot_2015-08-03-15-33-10

Tibolte commented 9 years ago

Good input, I'll try to improve what you pointed out.

Ashok-Varma commented 9 years ago

@Tibolte thankyou verymuch. Damn good library, and especially the support till 2.2 was great

Ashok-Varma commented 9 years ago

@Tibolte can you inform me once its fixed. I have nearly 50k users. So i can't implement it in my app until the bug is fixed

Tibolte commented 9 years ago

Sure you'll be notified here, I am doing it on my spare time, which is not easy to find these days ;) I hope to fix it by the end of the week.

Tibolte commented 9 years ago

I think what you observed is the view restoring its state, I didn't handle it. I'm currently working on a branch, I made some changes so we show the download button when restoring the view state. Can you just tell me what colors you want customizable on the view? That'll help me set-up a new release.

Ashok-Varma commented 9 years ago

@Tibolte 1) Download Icon color

public static Drawable getTintedDrawable(Drawable d, int color) { Drawable wrappedDrawable = DrawableCompat.wrap(d); DrawableCompat.setTint(wrappedDrawable, color); return wrappedDrawable; } You need latest support library to use that method

2) Color Surrounding the DownloadIcon,which is same as string color.

These are two main changes that need to be changed, If you wish you can also add for text color...etc

I am updating my app this weekend you can keep :- play.google.com/store/apps/details?id=com.ashokvarma.cocguidewiki Thanks to you, You decreased the amount of work i have to do

Ashok-Varma commented 9 years ago

You need latest support library to use that method

Tibolte commented 9 years ago

Allright, these are good ideas :+1: I'll see what I can do.

Tibolte commented 9 years ago

@Ashok-Varma sorry for the delay. I've been looking for solutions on how to dynamically change colors on path groups, this is harder than it looks. I'm wondering if that's really possible to do it. The only customizable color I can offer now is the background (green on your screenshots) behind the animated view. Let me know if those features can wait. I'd like to release the fix with restoring state.

Ashok-Varma commented 9 years ago

@Tibolte yeah you can release, the fix with restoring state, now. Those features can wait. And will this update solve #25 issue ??

Tibolte commented 9 years ago

Let me see if I can solve #25 today. I'll try to release something this evening. I'll probably open an issue regarding the colors separately.

Ashok-Varma commented 9 years ago

I will open issue regarding feature mark it as a feature.

Ashok-Varma commented 9 years ago

Issue opened @Tibolte

Tibolte commented 9 years ago

The fix for restoring view state is released under version 1.0.4, it will be available any time soon with Maven. Couldn't get time for #25.

Tibolte commented 9 years ago

1.0.4 is out, restoring state should be fixed then.