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

Rx search view #1

Open
wants to merge 20 commits into
base: rx-search-view
Choose a base branch
from

Conversation

danielgomezrico
Copy link

Merge code from master.

The idea is to continue this branch so it can be merged latter in JakeWharton#213.

kunny and others added 20 commits August 1, 2016 01:09
…laky

Fix RxSwipeDismissBehavior flaky test
Remove explicit 'public' modifier from Kotlin code.
This ensures that synchronous unsubscription (like take(1)) does not leak the listener onto the view.
Because other artifacts might depend on older versions of support-v4, force them to the current version so that normal resolution semantics apply properly.
Switch TargetApi for more correct RequiresApi annotation.
Add listeners after setting the unsubscribe action.
Depend only on the Core UI artifact from support-v4.
…rch-view

* 'master' of github.com:JakeWharton/RxBinding:
  Add BottomNavigationView binding
  Depend only on the Core UI artifact from support-v4.
  Add listeners after setting the unsubscribe action.
  Switch TargetApi for more correct RequiresApi annotation.
  Remove explicit 'public' modifier from Kotlin code.
  Add support for View key event binding
  change swipe speed to FAST to fix flaky test
  Guard against null in ViewEvent
  Add new SlidingPaneLayout bindings
  Fix order of modifiers in ViewEvent
  Added currentItem action to RxViewPager
  Add ViewPager page scroll state change binding

# Conflicts:
#	rxbinding-appcompat-v7-kotlin/src/main/kotlin/com/jakewharton/rxbinding/support/v7/widget/RxSearchView.kt
#	rxbinding-kotlin/src/main/kotlin/com/jakewharton/rxbinding/widget/RxSearchView.kt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants