Added tap speed check for example card onClick

[langsmith]

Resolves #1227 by adding a check of the tap speed on a single example card. Right now on master, tapping quickly will open a duplicate activity.

Before:

After:

[LukasPaczos]

This looks to me like a device-dependent way to resolve the issue. How about leveraging onPause/onResume to remove/add recycler click listener? Is the timing of those lifecycle callbacks correct?

[LukasPaczos]

First of all, there's a bug in the ItemClickSupport that it sets an irreversible click listener on the recycler view after the view is attached:

mapbox-android-demo/MapboxAndroidDemo/src/main/java/com/mapbox/mapboxandroiddemo/utils/ItemClickSupport.java

Lines 34 to 43 in 3dadd83

	private OnChildAttachStateChangeListener attachListener = new OnChildAttachStateChangeListener() {
	@Override
	public void onChildViewAttachedToWindow(View view) {
	if (onItemClickListener != null) {
	view.setOnClickListener(onClickListener);
	}
	if (onItemLongClickListener != null) {
	view.setOnLongClickListener(onLongClickListener);
	}
	}

Calling removeFrom after the recycler has been attached to the window has no effect at all, the view still has a set listener that's going to get notified. This whole setup should be refactored.

Currently, it can be worked around by removing the listener itself instead of the view:

// keep the reference to the created `ItemClickSupport`
itemClickSupport = ItemClickSupport.addTo(recyclerView);
itemClickSupport.setOnItemClickListener(this);

// when item is clicked, remove the listener
itemClickSupport.setOnItemClickListener(null);

// when activity is resumed, reattach the listener
itemClickSupport.setOnItemClickListener(this);

Locally I can confirm that this resolved the issue.

[LukasPaczos]

Is this enough to resolve the issue? I was testing with this invocation in onItemClicked.

[langsmith]

@Override
  protected void onPause() {
    super.onPause();
    itemClickSupport.setOnItemClickListener(null);
  }

is working 💯 for me

[LukasPaczos]

🚢

[LukasPaczos]

NIT: Since we are always setting the listener in the onResume, this is redundant.