searchablespinner icon indicating copy to clipboard operation
searchablespinner copied to clipboard

Why is the API 21 min required?

Open n0m0r3pa1n opened this issue 8 years ago • 11 comments

I saw that the only thing that makes API 21 min required is the circleRevealAnimation. You can use this lib instead: https://github.com/ozodrukh/CircularReveal Also, why do you need the constraint-layout lib. If you don't use it - remove it. The last one is if you can target version 15+ devices you also have to add a check for the removeOnGlobalLayout and removeGlobalOnLayout listeners.

n0m0r3pa1n avatar Jan 21 '17 17:01 n0m0r3pa1n

Hello @n0m0r3pa1n

You right about the constraint-layout lib and i remove it. Also, you right about the min sdk > 21. And i am considering to use the library you propose or rebound . But unfortunately the time is an issue and i do not know when i will do it. Finally, of course you are welcome to implement this issue and make a pull request.

Thanks

michaelprimez avatar Jan 25 '17 13:01 michaelprimez

Its having release time null pointer exception

SunilSDK avatar Mar 19 '17 19:03 SunilSDK

Please help me

SunilSDK avatar Mar 19 '17 19:03 SunilSDK

please remove the API 21 min, i want to use your awesome spinner.

domoemmanuel99 avatar Mar 22 '17 15:03 domoemmanuel99

Hi @sunilash143

I am not sure what you mean with "release time null pointer exception", you getting a null pointer exception only on the release version of your app?? Can you give me more infos please.

michaelprimez avatar Mar 22 '17 15:03 michaelprimez

Hi @domoemmanuel99

I am sorry this is not possible right now. The lib is based on code that require min sdk 21, and that kind of changes needs more time that i do not have right now. Of course, anyone is free to do it and create a pull request. Thanks

michaelprimez avatar Mar 22 '17 15:03 michaelprimez

Hi,

I totally get not having time to fix this, but my guess is that more people still target v15 as a minimum, so with this comment, I'm just saying that I'd also like the library to target v15 rather than v21, please?

Good luck!

enkemari avatar Apr 09 '17 17:04 enkemari

Hi @n0m0r3pa1n how to implement CircularReveal for PopupWindow in @michaelprimez searchablespinner library as it is giving me "java.lang.IllegalArgumentException: Parent must be instance of RevealViewGroup" error. Thank you.

idrisbohra avatar Oct 01 '17 12:10 idrisbohra

@michaelprimez PR based on comments from @n0m0r3pa1n https://github.com/michaelprimez/searchablespinner/pull/20

Akhil-Tripathi avatar Nov 11 '17 17:11 Akhil-Tripathi

@idrisbohra one soln to your question : https://github.com/michaelprimez/searchablespinner/pull/20

Akhil-Tripathi avatar Nov 11 '17 17:11 Akhil-Tripathi

you guys can modify minsdkversion and it will work

MansoorNisar92 avatar Jan 29 '19 13:01 MansoorNisar92