The app rate bar is definitely not customizable. The height of it also fixed. It is actually better to keep it short but I think there should not be a restriction.

I think other libraries for showing alerts can be used in this library and it can be great actually. Crouton can be used. It has to be customized a lot I think. SuperToasts may be better for this.

I think it will be more discreet :)

I can do it if it is ok. I want to discuss it here before integrating it.

Clicking on AppRate view in simulator (device which cannot handle "market" Intent) causes ActivityNotFoundException. try-catch should be used.

E/AndroidRuntime( 1395): android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.VIEW dat=market://details?id=com.mypackage } E/AndroidRuntime( 1395): at android.app.Instrumentation.checkStartActivityResult(Instrumentation.java:1632) E/AndroidRuntime( 1395): at android.app.Instrumentation.execStartActivity(Instrumentation.java:1424) E/AndroidRuntime( 1395): at android.app.Activity.startActivityForResult(Activity.java:3424) E/AndroidRuntime( 1395): at android.app.Activity.startActivityForResult(Activity.java:3385) E/AndroidRuntime( 1395): at android.support.v4.app.FragmentActivity.startActivityForResult(FragmentActivity.java:839) E/AndroidRuntime( 1395): at android.app.Activity.startActivity(Activity.java:3627) E/AndroidRuntime( 1395): at android.app.Activity.startActivity(Activity.java:3595) E/AndroidRuntime( 1395): at fr.nicolaspomepuy.discreetapprate.AppRate$2.onClick(AppRate.java:438) E/AndroidRuntime( 1395): at android.view.View.performClick(View.java:4438) E/AndroidRuntime( 1395): at android.view.View$PerformClick.run(View.java:18422) E/AndroidRuntime( 1395): at android.os.Handler.handleCallback(Handler.java:733) E/AndroidRuntime( 1395): at android.os.Handler.dispatchMessage(Handler.java:95) E/AndroidRuntime( 1395): at android.os.Looper.loop(Looper.java:136) E/AndroidRuntime( 1395): at android.app.ActivityThread.main(ActivityThread.java:5017) E/AndroidRuntime( 1395): at java.lang.reflect.Method.invokeNative(Native Method) E/AndroidRuntime( 1395): at java.lang.reflect.Method.invoke(Method.java:515) E/AndroidRuntime( 1395): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:779) E/AndroidRuntime( 1395): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595) E/AndroidRuntime( 1395): at dalvik.system.NativeStart.main(Native Method) W/ActivityManager( 447): Force finishing activity com.mypackage/com.mypackage.LiveViewActivity

If the user presses the rate view and opens the Play Store app and then presses HOME and then opens the initial app, it shows the Play Store app instead of showing the last screen before going to Play Store. To fix this, we need to use FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET.

Intent intent = new Intent(Intent.ACTION_VIEW, Uri.parse("market://details?id=" + activity.getPackageName())) intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET); activity.startaActivity(intent);

http://developer.android.com/reference/android/content/Intent.html#FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET

When i update from version 1.0.5 to 2.0.2, i am getting this:

[warn] [.../com.android.support-appcompat-v7-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk minSdkVersion='L'>: must be an integer number or codename.
[warn] [.../com.android.support-appcompat-v7-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk targetSdkVersion='L'>: must be an integer number or codename.
[warn] [.../com.android.support-support-v4-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk minSdkVersion='L'>: must be an integer number or codename.
[warn] [.../com.android.support-support-v4-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk targetSdkVersion='L'>: must be an integer number or codename.
[info] Performing full resource merge
java.lang.RuntimeException: Manifest merging failed. See console for more info.
        at com.android.builder.core.AndroidBuilder.doMerge(AndroidBuilder.java:920)
        at com.android.builder.core.AndroidBuilder.mergeLibraryManifests(AndroidBuilder.java:906)
        at com.android.builder.core.AndroidBuilder.processManifest(AndroidBuilder.java:612)
        at android.Tasks$$anonfun$58.apply(tasks.scala:912)
        at android.Tasks$$anonfun$58.apply(tasks.scala:899)
        at scala.Function9$$anonfun$tupled$1.apply(Function9.scala:35)
        at scala.Function9$$anonfun$tupled$1.apply(Function9.scala:34)
        at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
        at sbt.$tilde$greater$$anonfun$$u2219$1.apply(TypeFunctions.scala:42)
        at sbt.std.Transform$$anon$4.work(System.scala:64)
        at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:237)
        at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:237)
        at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:18)
        at sbt.Execute.work(Execute.scala:244)
        at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:237)
        at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:237)
        at sbt.ConcurrentRestrictions$$anon$4$$anonfun$1.apply(ConcurrentRestrictions.scala:160)
        at sbt.CompletionService$$anon$2.call(CompletionService.scala:30)
        at java.util.concurrent.FutureTask.run(FutureTask.java:262)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
        at java.util.concurrent.FutureTask.run(FutureTask.java:262)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:745)
[error] (android:processManifest) Manifest merging failed. See console for more info.

It seems like this library is pulling in the latest android support revisions.

Could it be that the "+" is causing it to do that?

dependencies {
compile 'com.android.support:appcompat-v7:+'
}

Looks like custom views are pretty undocumented. Looks like the lib searches for specific IDs. And also it overrides the text on the "rate" button. Custom views are not useable this way

Like the user suggested in issue #13, I also just copy the source into my project. This means that I have to manually merge strings.xml.

Please could you rename strings.xml so that I can just copy as is.

For example discreet-app-rate.xml

I get the following exception report:

java.lang.NoSuchMethodError: android.view.View.setBackground
at fr.nicolaspomepuy.discreetapprate.AppRate.showAppRate(AppRate.java:568)
at fr.nicolaspomepuy.discreetapprate.AppRate.checkAndShow(AppRate.java:355)

To get rid of it I set the following two lines to use setBackgroundDrawable method (although I know it might not be the best fix):

close.setBackground(activity.getResources().getDrawable(R.drawable.selectable_button_light));

close.setBackground(activity.getResources().getDrawable(R.drawable.selectable_button_dark));

The library uses reflection to get "firstInstallTime" (PackageInfo) which is available for API 9 and above. However for API 8 and below AppRate will generate NullPointerException on calling checkAndShow() method.

// AppRate.java Date installDate = Utils.installTimeFromPackageManager(activity.getPackageManager(), activity.getPackageName()); Date now = new Date(); if (now.getTime() - installDate.getTime() < installedSince * 1000) { // <<<<<<<<<< NullPointerException for Android 2.2 and below!!!!

// Utils.java public static Date installTimeFromPackageManager( PackageManager packageManager, String packageName) { .... // field wasn't found return null; }

AppRate check application network state and try to avoid not to be seen in offline state. In this way, users will see AppRate only when they can connect Google Play Store.

Create a new method to get the ActionBarHeight --> Move this method to the Utils like your method getStatusBarHeight

With the previous attempt to get the ActionBarSize it caused some issue if the TranslucentDecorView was used. See the screenshot on the bottom.

Now it should work in 90% of the cases. It will still cause issues if someone has a custom-rom with a different ActionBarSize.

For my private projects i use the DrawInsetsLayout but we can't use it here.

Master branch

java.lang.NoSuchMethodError: android.content.SharedPreferences$Editor.apply at fr.nicolaspomepuy.discreetapprate.AppRate.commitEditor(SourceFile:652) at fr.nicolaspomepuy.discreetapprate.AppRate.incrementViews(SourceFile:464) at fr.nicolaspomepuy.discreetapprate.AppRate.checkAndShow(SourceFile:320)

The error occurs in the code:

private void commitEditor() {
    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.FROYO) {
        editor.apply();
    } else {
        editor.commit();
    }
}

Change Build.VERSION_CODES.FROYO to Build.VERSION_CODES.GINGERBREAD.

Allows you to prompt the user to enter a star rating, and optionally listen for the rating they provide and perform different actions on whether it was above or below a threshold.

Also, updated build tools and support lib version.

Apologies for posting my question here as it is not really an issue and I didn't find a better way to contact the author.

Is it possible to have "nested" navigation in snackbar, i.e. first display : do you like app : yes / no and then depending on action -> "snackbar" displays next set of options, with logic similar to this

Also, Am I right in assumption that layout of "snackBar" is fully customized ?

Something that would be a 2 line snackbar - with Line 1 being : Where did you hear about us? Line 2 : A) Facebook B) Twitter C) others, and user can click on one of the 3 actions

Steps to reproduce:

1. Run in emulator or any other device which do not have Google Play installed.
2. Call forceShow(). The apprate is shown.
3. Press on AppRate to get a crash android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.VIEW dat=market://details?id=com.myproject flg=0x10000000 }

Fix 1: In forceShow() should be if (!Utils.isGooglePlayInstalled(activity)) { if (debug) LogD("Play Store is not installed. Won't do anything"); return; }

instead of if (!Utils.isGooglePlayInstalled(activity)) { if (debug) LogD("Play Store is not installed. Won't do anything"); }

Fix 2: Use try {} catch (Exception e) {} in onClick() while calling Intent intent = new Intent(Intent.ACTION_VIEW, Uri.parse("market://details?id=" + packageName)); intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK); activity.startActivity(intent);