Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4f8e451d authored by Tadashi G. Takaoka's avatar Tadashi G. Takaoka
Browse files

Show gesture floating preview text is off by default

Bug: 7043127
Change-Id: Icabf080f43a5934fef3824919bb03229522d5e5a
parent 63dd5b4c
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -138,11 +138,11 @@
                android:persistent="true"
                android:defaultValue="true" />
            <CheckBoxPreference
                android:key="pref_gesture_floating_preview_text"
                android:key="pref_show_gesture_floating_preview_text"
                android:title="@string/gesture_floating_preview_text"
                android:summary="@string/gesture_floating_preview_text_summary"
                android:persistent="true"
                android:defaultValue="true" />
                android:defaultValue="false" />
        </PreferenceScreen>
    </PreferenceCategory>
</PreferenceScreen>
+20 −12
Original line number Diff line number Diff line
@@ -189,6 +189,14 @@ public class PreviewPlacerView extends RelativeLayout {
    }

    public void invalidatePointer(final PointerTracker tracker, final boolean isOldestTracker) {
        final boolean needsToUpdateLastPointer =
                isOldestTracker && mDrawsGestureFloatingPreviewText;
        if (needsToUpdateLastPointer) {
            mLastPointerX = tracker.getLastX();
            mLastPointerY = tracker.getLastY();
        }

        if (mDrawsGesturePreviewTrail) {
            GesturePreviewTrail trail;
            synchronized (mGesturePreviewTrails) {
                trail = mGesturePreviewTrails.get(tracker.mPointerId);
@@ -198,14 +206,13 @@ public class PreviewPlacerView extends RelativeLayout {
                }
            }
            trail.addStroke(tracker.getGestureStrokeWithPreviewTrail(), tracker.getDownTime());

        if (isOldestTracker) {
            mLastPointerX = tracker.getLastX();
            mLastPointerY = tracker.getLastY();
        }

        // TODO: Should narrow the invalidate region.
        if (mDrawsGesturePreviewTrail || needsToUpdateLastPointer) {
            invalidate();
        }
    }

    @Override
    protected void onDetachedFromWindow() {
@@ -262,6 +269,7 @@ public class PreviewPlacerView extends RelativeLayout {
    }

    public void setGestureFloatingPreviewText(final String gestureFloatingPreviewText) {
        if (!mDrawsGestureFloatingPreviewText) return;
        mGestureFloatingPreviewText = gestureFloatingPreviewText;
        invalidate();
    }
+4 −4
Original line number Diff line number Diff line
@@ -77,8 +77,8 @@ public class Settings extends InputMethodSettingsFragment
    public static final String PREF_KEYPRESS_SOUND_VOLUME =
            "pref_keypress_sound_volume";
    public static final String PREF_GESTURE_PREVIEW_TRAIL = "pref_gesture_preview_trail";
    public static final String PREF_GESTURE_FLOATING_PREVIEW_TEXT =
            "pref_gesture_floating_preview_text";
    public static final String PREF_SHOW_GESTURE_FLOATING_PREVIEW_TEXT =
            "pref_show_gesture_floating_preview_text";

    public static final String PREF_INPUT_LANGUAGE = "input_language";
    public static final String PREF_SELECTED_LANGUAGES = "selected_languages";
@@ -207,7 +207,7 @@ public class Settings extends InputMethodSettingsFragment
                R.bool.config_gesture_input_enabled_by_build_config);
        final Preference gesturePreviewTrail = findPreference(PREF_GESTURE_PREVIEW_TRAIL);
        final Preference gestureFloatingPreviewText = findPreference(
                PREF_GESTURE_FLOATING_PREVIEW_TEXT);
                PREF_SHOW_GESTURE_FLOATING_PREVIEW_TEXT);
        if (!gestureInputEnabledByBuildConfig) {
            miscSettings.removePreference(findPreference(PREF_GESTURE_INPUT));
            miscSettings.removePreference(gesturePreviewTrail);
@@ -304,7 +304,7 @@ public class Settings extends InputMethodSettingsFragment
                        PREF_GESTURE_INPUT, true);
                setPreferenceEnabled(findPreference(PREF_GESTURE_PREVIEW_TRAIL),
                        gestureInputEnabledByUser);
                setPreferenceEnabled(findPreference(PREF_GESTURE_FLOATING_PREVIEW_TEXT),
                setPreferenceEnabled(findPreference(PREF_SHOW_GESTURE_FLOATING_PREVIEW_TEXT),
                        gestureInputEnabledByUser);
            }
        }
+1 −1
Original line number Diff line number Diff line
@@ -178,7 +178,7 @@ public final class SettingsValues {
                && prefs.getBoolean(Settings.PREF_GESTURE_INPUT, true);
        mGesturePreviewTrailEnabled = prefs.getBoolean(Settings.PREF_GESTURE_PREVIEW_TRAIL, true);
        mGestureFloatingPreviewTextEnabled = prefs.getBoolean(
                Settings.PREF_GESTURE_FLOATING_PREVIEW_TEXT, true);
                Settings.PREF_SHOW_GESTURE_FLOATING_PREVIEW_TEXT, false);
        mCorrectionEnabled = mAutoCorrectEnabled && !mInputAttributes.mInputTypeNoAutoCorrect;
        mSuggestionVisibility = createSuggestionVisibility(res);
    }