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

[fix] multiple touch #200

Merged
merged 5 commits into from
Sep 26, 2023
Merged

[fix] multiple touch #200

merged 5 commits into from
Sep 26, 2023

Conversation

Dan2dani
Copy link
Member

Related issue πŸ› 

Work Description ✏️

  • λ²„νŠΌμ΄ λΉ λ₯Έ μ‹œκ°„λ‚΄μ— μ—¬λŸ¬λ²ˆ ν΄λ¦­λ˜λ©΄μ„œ λ°œμƒν•˜λŠ” μ„œλ²„μ—λŸ¬ 등을 ν•΄κ²°ν•©λ‹ˆλ‹€.

Copy link
Collaborator

@jihyunniiii jihyunniiii left a comment

Choose a reason for hiding this comment

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

λ„ˆ μ§„μ«˜ λŠ₯λ ₯μžλ‹€

Comment on lines 44 to 50
ViewPager2.OnPageChangeCallback() {
override fun onPageSelected(position: Int) {
super.onPageSelected(position)
viewModel.setCurrentPage(position)
}
})
ViewPager2.OnPageChangeCallback() {
override fun onPageSelected(position: Int) {
super.onPageSelected(position)
viewModel.setCurrentPage(position)
}
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

ktlint 였λ₯˜ λ‚˜λŠ” κ±° μ—¬κΈ° λ•Œλ¬Έμž„μš” ν•œ μΉΈμ”© νƒ­ λˆŒλŸ¬μ€˜μ•Όν•¨

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅐ이거 μ§„μ§œ 화남..

Comment on lines +11 to +25
inline fun View.setOnSingleClickListener(
delay: Long = 500L,
crossinline block: (View) -> Unit
) {
var isClickable = true
setOnClickListener { view ->
if (isClickable) {
isClickable = false
block(view)
view.postDelayed({
isClickable = true
}, delay)
}
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

κ°œμ©λ‹€ ν΄λ¦­ν•˜κ³  λ”œλ ˆμ΄ μ€˜μ„œ κ·Έ μ‹œκ°„ λ™μ•ˆ 클릭 μ•ˆ λ˜λ„λ‘ ν•˜λŠ” κ±° 맞음??

@Dan2dani Dan2dani changed the title Fix multiple touch [fix] multiple touch Sep 26, 2023
@Dan2dani Dan2dani merged commit c27c819 into develop Sep 26, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[fix] λ²„νŠΌ 연타 막기
2 participants