Skip to content

Improved vibration feedback on both ACTION_DOWN and ACTION_UP #577

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.

Sign up for GitHub

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

Open
Manash396 wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Improved vibration feedback on both ACTION_DOWN and ACTION_UP #577

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
190348b
Improved vibration feedback on both ACTION_DOWN and ACTION_UP
Manash396 Jul 18, 2025
8c2de6c
resolved the conflicts, improved vibration feedback commit
Manash396 Jul 18, 2025
d62fbb6
fix the issues mentioned pr comment
Manash396 Sep 22, 2025
d5b131a
Merge branch 'main' of https://github.com/Manash396/OpenCalc into imp…
Manash396 Sep 22, 2025
1e7903d
Merge branch 'clementwzk:main' into improved-vibration-feedback
Manash396 Sep 29, 2025
ee8cb5c
Merge branch 'main' of https://github.com/Manash396/OpenCalc into imp…
Manash396 Sep 29, 2025
91bbfb2
Merge branch 'improved-vibration-feedback' of https://github.com/Mana…
Manash396 Sep 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
64 changes: 62 additions & 2 deletions app/src/main/java/com/darkempire78/opencalculator/activities/MainActivity.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import android.text.TextWatcher
import android.util.Log
import android.view.HapticFeedbackConstants
import android.view.MenuItem
import android.view.MotionEvent
import android.view.View
import android.view.WindowManager
import android.view.accessibility.AccessibilityEvent
Expand Down Expand Up @@ -98,6 +99,7 @@ class MainActivity : AppCompatActivity() {
WindowManager.LayoutParams.FLAG_TURN_SCREEN_ON
)


// Themes
val themes = Themes(this)
themes.applyDayNightOverride()
Expand All @@ -112,6 +114,9 @@ class MainActivity : AppCompatActivity() {
view = binding.root
setContentView(view)

// set buttons on touch listener to get feedback vibration on touch
setOnBottontouchlistener()

// Disable the keyboard on display EditText
binding.input.showSoftInputOnFocus = false

Expand Down Expand Up @@ -320,6 +325,61 @@ class MainActivity : AppCompatActivity() {

}

private fun setOnBottontouchlistener() {
applyVibrationToKey(binding.squareButton)
applyVibrationToKey(binding.piButton)
applyVibrationToKey(binding.exponentButton)
applyVibrationToKey(binding.factorialButton)
applyVibrationToKey(binding.squareButton)
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate call to applyVibrationToKey(binding.squareButton). This button is configured twice on lines 329 and 333.

Suggested change
applyVibrationToKey(binding.squareButton)

Copilot uses AI. Check for mistakes.
applyVibrationToKey(binding.piButton)
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate call to applyVibrationToKey(binding.piButton). This button is configured twice on lines 330 and 334.

Suggested change
applyVibrationToKey(binding.piButton)

Copilot uses AI. Check for mistakes.
applyVibrationToKey(binding.exponentButton)
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate call to applyVibrationToKey(binding.exponentButton). This button is configured twice on lines 331 and 335.

Suggested change
applyVibrationToKey(binding.exponentButton)

Copilot uses AI. Check for mistakes.
applyVibrationToKey(binding.factorialButton)
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate call to applyVibrationToKey(binding.factorialButton). This button is configured twice on lines 332 and 336.

Suggested change
applyVibrationToKey(binding.factorialButton)

Copilot uses AI. Check for mistakes.
applyVibrationToKey(binding.degreeButton)
applyVibrationToKey(binding.sineButton)
applyVibrationToKey(binding.cosineButton)
applyVibrationToKey(binding.tangentButton)
applyVibrationToKey(binding.invButton)
applyVibrationToKey(binding.eButton)
applyVibrationToKey(binding.naturalLogarithmButton)
applyVibrationToKey(binding.logarithmButton)
binding.log2Button?.let { applyVibrationToKey(it) }
applyVibrationToKey(binding.clearButton)
applyVibrationToKey(binding.leftParenthesisButton)
applyVibrationToKey(binding.rightParenthesisButton)
applyVibrationToKey(binding.parenthesesButton)
applyVibrationToKey(binding.divideBy100Button)
applyVibrationToKey(binding.divideButton)
applyVibrationToKey(binding.sevenButton)
applyVibrationToKey(binding.eightButton)
applyVibrationToKey(binding.nineButton)
applyVibrationToKey(binding.multiplyButton)
applyVibrationToKey(binding.fourButton)
applyVibrationToKey(binding.fiveButton)
applyVibrationToKey(binding.sixButton)
applyVibrationToKey(binding.subtractButton)
applyVibrationToKey(binding.oneButton)
applyVibrationToKey(binding.twoButton)
applyVibrationToKey(binding.threeButton)
applyVibrationToKey(binding.addButton)
applyVibrationToKey(binding.zeroButton)
applyVibrationToKey(binding.pointButton)
applyVibrationToKey(binding.backspaceButton)
applyVibrationToKey(binding.equalsButton)



}

private fun applyVibrationToKey(button: View) {
button.setOnTouchListener { v, event ->
if (event.action == MotionEvent.ACTION_DOWN) {
keyVibration(v)
}
false // return false to allow normal click to continue
}
}


// Displays a popup menu with options to insert double zeros ("00") or triple zeros ("000") into the specified EditText when the zero button is long-pressed.
private fun showPopupMenu(zeroButton: Button) {
val popupMenu = PopupMenu(this, zeroButton)
Expand Down Expand Up @@ -842,8 +902,8 @@ class MainActivity : AppCompatActivity() {
&& (binding.input.text[endPosition].isDigit()
|| binding.input.text[endPosition].toString() == decimalSeparatorSymbol
|| binding.input.text[endPosition].toString() == groupingSeparatorSymbol)) {
endPosition += 1
}
endPosition += 1
}
}
currentNumber = binding.input.text.substring(startPosition, endPosition)
}
Expand Down