Commit 846c6ef2 authored by Leonardo Aramaki's avatar Leonardo Aramaki

Remove unnecessary callbacks and fixed keyboard closing behavior

parent 18800044
#Thu Feb 15 15:50:42 BRST 2018
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-4.1-all.zip
#!/usr/bin/env bash
##############################################################################
##
## Gradle start up script for UN*X
##
##############################################################################
# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
DEFAULT_JVM_OPTS=""
APP_NAME="Gradle"
APP_BASE_NAME=`basename "$0"`
# Use the maximum available, or set MAX_FD != -1 to use that value.
MAX_FD="maximum"
warn ( ) {
echo "$*"
}
die ( ) {
echo
echo "$*"
echo
exit 1
}
# OS specific support (must be 'true' or 'false').
cygwin=false
msys=false
darwin=false
case "`uname`" in
CYGWIN* )
cygwin=true
;;
Darwin* )
darwin=true
;;
MINGW* )
msys=true
;;
esac
# Attempt to set APP_HOME
# Resolve links: $0 may be a link
PRG="$0"
# Need this for relative symlinks.
while [ -h "$PRG" ] ; do
ls=`ls -ld "$PRG"`
link=`expr "$ls" : '.*-> \(.*\)$'`
if expr "$link" : '/.*' > /dev/null; then
PRG="$link"
else
PRG=`dirname "$PRG"`"/$link"
fi
done
SAVED="`pwd`"
cd "`dirname \"$PRG\"`/" >/dev/null
APP_HOME="`pwd -P`"
cd "$SAVED" >/dev/null
CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar
# Determine the Java command to use to start the JVM.
if [ -n "$JAVA_HOME" ] ; then
if [ -x "$JAVA_HOME/jre/sh/java" ] ; then
# IBM's JDK on AIX uses strange locations for the executables
JAVACMD="$JAVA_HOME/jre/sh/java"
else
JAVACMD="$JAVA_HOME/bin/java"
fi
if [ ! -x "$JAVACMD" ] ; then
die "ERROR: JAVA_HOME is set to an invalid directory: $JAVA_HOME
Please set the JAVA_HOME variable in your environment to match the
location of your Java installation."
fi
else
JAVACMD="java"
which java >/dev/null 2>&1 || die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
Please set the JAVA_HOME variable in your environment to match the
location of your Java installation."
fi
# Increase the maximum file descriptors if we can.
if [ "$cygwin" = "false" -a "$darwin" = "false" ] ; then
MAX_FD_LIMIT=`ulimit -H -n`
if [ $? -eq 0 ] ; then
if [ "$MAX_FD" = "maximum" -o "$MAX_FD" = "max" ] ; then
MAX_FD="$MAX_FD_LIMIT"
fi
ulimit -n $MAX_FD
if [ $? -ne 0 ] ; then
warn "Could not set maximum file descriptor limit: $MAX_FD"
fi
else
warn "Could not query maximum file descriptor limit: $MAX_FD_LIMIT"
fi
fi
# For Darwin, add options to specify how the application appears in the dock
if $darwin; then
GRADLE_OPTS="$GRADLE_OPTS \"-Xdock:name=$APP_NAME\" \"-Xdock:icon=$APP_HOME/media/gradle.icns\""
fi
# For Cygwin, switch paths to Windows format before running java
if $cygwin ; then
APP_HOME=`cygpath --path --mixed "$APP_HOME"`
CLASSPATH=`cygpath --path --mixed "$CLASSPATH"`
JAVACMD=`cygpath --unix "$JAVACMD"`
# We build the pattern for arguments to be converted via cygpath
ROOTDIRSRAW=`find -L / -maxdepth 1 -mindepth 1 -type d 2>/dev/null`
SEP=""
for dir in $ROOTDIRSRAW ; do
ROOTDIRS="$ROOTDIRS$SEP$dir"
SEP="|"
done
OURCYGPATTERN="(^($ROOTDIRS))"
# Add a user-defined pattern to the cygpath arguments
if [ "$GRADLE_CYGPATTERN" != "" ] ; then
OURCYGPATTERN="$OURCYGPATTERN|($GRADLE_CYGPATTERN)"
fi
# Now convert the arguments - kludge to limit ourselves to /bin/sh
i=0
for arg in "$@" ; do
CHECK=`echo "$arg"|egrep -c "$OURCYGPATTERN" -`
CHECK2=`echo "$arg"|egrep -c "^-"` ### Determine if an option
if [ $CHECK -ne 0 ] && [ $CHECK2 -eq 0 ] ; then ### Added a condition
eval `echo args$i`=`cygpath --path --ignore --mixed "$arg"`
else
eval `echo args$i`="\"$arg\""
fi
i=$((i+1))
done
case $i in
(0) set -- ;;
(1) set -- "$args0" ;;
(2) set -- "$args0" "$args1" ;;
(3) set -- "$args0" "$args1" "$args2" ;;
(4) set -- "$args0" "$args1" "$args2" "$args3" ;;
(5) set -- "$args0" "$args1" "$args2" "$args3" "$args4" ;;
(6) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" ;;
(7) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" ;;
(8) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" ;;
(9) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" "$args8" ;;
esac
fi
# Split up the JVM_OPTS And GRADLE_OPTS values into an array, following the shell quoting and substitution rules
function splitJvmOpts() {
JVM_OPTS=("$@")
}
eval splitJvmOpts $DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS
JVM_OPTS[${#JVM_OPTS[*]}]="-Dorg.gradle.appname=$APP_BASE_NAME"
exec "$JAVACMD" "${JVM_OPTS[@]}" -classpath "$CLASSPATH" org.gradle.wrapper.GradleWrapperMain "$@"
@if "%DEBUG%" == "" @echo off
@rem ##########################################################################
@rem
@rem Gradle startup script for Windows
@rem
@rem ##########################################################################
@rem Set local scope for the variables with windows NT shell
if "%OS%"=="Windows_NT" setlocal
@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
set DEFAULT_JVM_OPTS=
set DIRNAME=%~dp0
if "%DIRNAME%" == "" set DIRNAME=.
set APP_BASE_NAME=%~n0
set APP_HOME=%DIRNAME%
@rem Find java.exe
if defined JAVA_HOME goto findJavaFromJavaHome
set JAVA_EXE=java.exe
%JAVA_EXE% -version >NUL 2>&1
if "%ERRORLEVEL%" == "0" goto init
echo.
echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
echo.
echo Please set the JAVA_HOME variable in your environment to match the
echo location of your Java installation.
goto fail
:findJavaFromJavaHome
set JAVA_HOME=%JAVA_HOME:"=%
set JAVA_EXE=%JAVA_HOME%/bin/java.exe
if exist "%JAVA_EXE%" goto init
echo.
echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%
echo.
echo Please set the JAVA_HOME variable in your environment to match the
echo location of your Java installation.
goto fail
:init
@rem Get command-line arguments, handling Windowz variants
if not "%OS%" == "Windows_NT" goto win9xME_args
if "%@eval[2+2]" == "4" goto 4NT_args
:win9xME_args
@rem Slurp the command line arguments.
set CMD_LINE_ARGS=
set _SKIP=2
:win9xME_args_slurp
if "x%~1" == "x" goto execute
set CMD_LINE_ARGS=%*
goto execute
:4NT_args
@rem Get arguments from the 4NT Shell from JP Software
set CMD_LINE_ARGS=%$
:execute
@rem Setup the command line
set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
@rem Execute Gradle
"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %CMD_LINE_ARGS%
:end
@rem End local scope for the variables with windows NT shell
if "%ERRORLEVEL%"=="0" goto mainEnd
:fail
rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of
rem the _cmd.exe /c_ return code!
if not "" == "%GRADLE_EXIT_CONSOLE%" exit 1
exit /b 1
:mainEnd
if "%OS%"=="Windows_NT" endlocal
:omega
...@@ -45,7 +45,7 @@ ...@@ -45,7 +45,7 @@
<activity <activity
android:name=".chatroom.ui.ChatRoomActivity" android:name=".chatroom.ui.ChatRoomActivity"
android:windowSoftInputMode="adjustPan" android:windowSoftInputMode="adjustNothing"
android:theme="@style/AppTheme" /> android:theme="@style/AppTheme" />
<activity <activity
......
...@@ -15,6 +15,7 @@ import android.support.v7.widget.LinearLayoutManager ...@@ -15,6 +15,7 @@ import android.support.v7.widget.LinearLayoutManager
import android.support.v7.widget.RecyclerView import android.support.v7.widget.RecyclerView
import android.text.method.ScrollingMovementMethod import android.text.method.ScrollingMovementMethod
import android.view.* import android.view.*
import android.widget.ImageButton
import chat.rocket.android.R import chat.rocket.android.R
import chat.rocket.android.chatroom.presentation.ChatRoomPresenter import chat.rocket.android.chatroom.presentation.ChatRoomPresenter
import chat.rocket.android.chatroom.presentation.ChatRoomView import chat.rocket.android.chatroom.presentation.ChatRoomView
...@@ -27,7 +28,6 @@ import chat.rocket.android.widget.emoji.ComposerEditText ...@@ -27,7 +28,6 @@ import chat.rocket.android.widget.emoji.ComposerEditText
import chat.rocket.android.widget.emoji.Emoji import chat.rocket.android.widget.emoji.Emoji
import chat.rocket.android.widget.emoji.EmojiFragment import chat.rocket.android.widget.emoji.EmojiFragment
import chat.rocket.android.widget.emoji.EmojiParser import chat.rocket.android.widget.emoji.EmojiParser
import chat.rocket.common.util.ifNull
import dagger.android.support.AndroidSupportInjection import dagger.android.support.AndroidSupportInjection
import kotlinx.android.synthetic.main.fragment_chat_room.* import kotlinx.android.synthetic.main.fragment_chat_room.*
import kotlinx.android.synthetic.main.message_attachment_options.* import kotlinx.android.synthetic.main.message_attachment_options.*
...@@ -247,24 +247,6 @@ class ChatRoomFragment : Fragment(), ChatRoomView, EmojiFragment.EmojiKeyboardLi ...@@ -247,24 +247,6 @@ class ChatRoomFragment : Fragment(), ChatRoomView, EmojiFragment.EmojiKeyboardLi
} }
} }
override fun onSoftKeyboardHidden() {
setReactionButtonIcon(R.drawable.ic_keyboard_black_24dp)
}
override fun onSoftKeyboardShown() {
setReactionButtonIcon(R.drawable.ic_reaction_24dp)
recycler_view.scrollToPosition(0)
}
override fun onEmojiKeyboardHidden() {
setReactionButtonIcon(R.drawable.ic_reaction_24dp)
}
override fun onEmojiKeyboardShown() {
setReactionButtonIcon(R.drawable.ic_keyboard_black_24dp)
recycler_view.scrollToPosition(0)
}
private fun setReactionButtonIcon(@DrawableRes drawableId: Int) { private fun setReactionButtonIcon(@DrawableRes drawableId: Int) {
button_add_reaction.setImageResource(drawableId) button_add_reaction.setImageResource(drawableId)
button_add_reaction.setTag(drawableId) button_add_reaction.setTag(drawableId)
...@@ -301,8 +283,19 @@ class ChatRoomFragment : Fragment(), ChatRoomView, EmojiFragment.EmojiKeyboardLi ...@@ -301,8 +283,19 @@ class ChatRoomFragment : Fragment(), ChatRoomView, EmojiFragment.EmojiKeyboardLi
}) })
text_message.listener = object : ComposerEditText.ComposerEditTextListener { text_message.listener = object : ComposerEditText.ComposerEditTextListener {
override fun onKeyboardClose() { override fun onKeyboardOpened() {
activity?.let { activity?.let {
val fragment = EmojiFragment.getOrAttach(it, R.id.emoji_fragment_placeholder, composer)
if (fragment.isCollapsed()) {
fragment.show()
}
setReactionButtonIcon(R.drawable.ic_reaction_24dp)
}
}
override fun onKeyboardClosed() {
activity?.let {
setReactionButtonIcon(R.drawable.ic_reaction_24dp)
val fragment = EmojiFragment.getOrAttach(it, R.id.emoji_fragment_placeholder, composer) val fragment = EmojiFragment.getOrAttach(it, R.id.emoji_fragment_placeholder, composer)
if (fragment.isCollapsed()) { if (fragment.isCollapsed()) {
it.onBackPressed() it.onBackPressed()
...@@ -358,8 +351,12 @@ class ChatRoomFragment : Fragment(), ChatRoomView, EmojiFragment.EmojiKeyboardLi ...@@ -358,8 +351,12 @@ class ChatRoomFragment : Fragment(), ChatRoomView, EmojiFragment.EmojiKeyboardLi
if (!emojiFragment.isShown()) { if (!emojiFragment.isShown()) {
emojiFragment.show() emojiFragment.show()
} }
setReactionButtonIcon(R.drawable.ic_keyboard_black_24dp)
}
R.drawable.ic_keyboard_black_24dp -> {
KeyboardHelper.showSoftKeyboard(editor)
setReactionButtonIcon(R.drawable.ic_reaction_24dp)
} }
R.drawable.ic_keyboard_black_24dp -> KeyboardHelper.showSoftKeyboard(editor)
} }
} }
} }
......
...@@ -4,6 +4,7 @@ import android.content.Context ...@@ -4,6 +4,7 @@ import android.content.Context
import android.support.v7.widget.AppCompatEditText import android.support.v7.widget.AppCompatEditText
import android.util.AttributeSet import android.util.AttributeSet
import android.view.KeyEvent import android.view.KeyEvent
import android.view.MotionEvent
class ComposerEditText : AppCompatEditText { class ComposerEditText : AppCompatEditText {
var listener: ComposerEditTextListener? = null var listener: ComposerEditTextListener? = null
...@@ -21,7 +22,7 @@ class ComposerEditText : AppCompatEditText { ...@@ -21,7 +22,7 @@ class ComposerEditText : AppCompatEditText {
if (state != null) { if (state != null) {
if (event.action == KeyEvent.ACTION_DOWN) { if (event.action == KeyEvent.ACTION_DOWN) {
state.startTracking(event, this) state.startTracking(event, this)
listener?.onKeyboardClose() listener?.onKeyboardClosed()
} }
return true return true
} }
...@@ -29,7 +30,13 @@ class ComposerEditText : AppCompatEditText { ...@@ -29,7 +30,13 @@ class ComposerEditText : AppCompatEditText {
return super.dispatchKeyEventPreIme(event) return super.dispatchKeyEventPreIme(event)
} }
override fun performClick(): Boolean {
listener?.onKeyboardOpened()
return super.performClick()
}
interface ComposerEditTextListener { interface ComposerEditTextListener {
fun onKeyboardClose() fun onKeyboardClosed()
fun onKeyboardOpened()
} }
} }
\ No newline at end of file
...@@ -103,7 +103,6 @@ class EmojiFragment : Fragment() { ...@@ -103,7 +103,6 @@ class EmojiFragment : Fragment() {
setKeyboardHeight(currentKeyboardHeight) setKeyboardHeight(currentKeyboardHeight)
softKeyboardVisible = true softKeyboardVisible = true
openHidden() openHidden()
listener?.onSoftKeyboardShown()
} else if (lastVisibleDecorViewHeight + MIN_KEYBOARD_HEIGHT_PX < visibleDecorViewHeight) { } else if (lastVisibleDecorViewHeight + MIN_KEYBOARD_HEIGHT_PX < visibleDecorViewHeight) {
// Notify listener about keyboard being hidden. // Notify listener about keyboard being hidden.
softKeyboardVisible = false softKeyboardVisible = false
...@@ -122,22 +121,6 @@ class EmojiFragment : Fragment() { ...@@ -122,22 +121,6 @@ class EmojiFragment : Fragment() {
} }
viewPager.adapter = CategoryPagerAdapter(object : EmojiKeyboardListener { viewPager.adapter = CategoryPagerAdapter(object : EmojiKeyboardListener {
override fun onEmojiKeyboardShown() {
// Do nothing here.
}
override fun onEmojiKeyboardHidden() {
// Do nothing here.
}
override fun onSoftKeyboardHidden() {
// Do nothing here.
}
override fun onSoftKeyboardShown() {
// Do nothing here.
}
override fun onEmojiAdded(emoji: Emoji) { override fun onEmojiAdded(emoji: Emoji) {
EmojiRepository.addToRecents(emoji) EmojiRepository.addToRecents(emoji)
callback.onEmojiAdded(emoji) callback.onEmojiAdded(emoji)
...@@ -214,7 +197,6 @@ class EmojiFragment : Fragment() { ...@@ -214,7 +197,6 @@ class EmojiFragment : Fragment() {
*/ */
fun show() { fun show() {
parentContainer.setVisible(true) parentContainer.setVisible(true)
listener?.onEmojiKeyboardShown()
} }
fun openHidden() { fun openHidden() {
...@@ -229,7 +211,6 @@ class EmojiFragment : Fragment() { ...@@ -229,7 +211,6 @@ class EmojiFragment : Fragment() {
// when the emoji one is about to get close. Hence we should invoke our listener to update // when the emoji one is about to get close. Hence we should invoke our listener to update
// the UI as if the soft keyboard is hidden. // the UI as if the soft keyboard is hidden.
parentContainer.setVisible(false) parentContainer.setVisible(false)
listener?.onEmojiKeyboardHidden()
} }
/** /**
...@@ -253,13 +234,5 @@ class EmojiFragment : Fragment() { ...@@ -253,13 +234,5 @@ class EmojiFragment : Fragment() {
* @param emoji The selected emoji * @param emoji The selected emoji
*/ */
fun onEmojiAdded(emoji: Emoji) fun onEmojiAdded(emoji: Emoji)
fun onSoftKeyboardHidden()
fun onSoftKeyboardShown()
fun onEmojiKeyboardHidden()
fun onEmojiKeyboardShown()
} }
} }
\ No newline at end of file
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment