GitHawk icon indicating copy to clipboard operation
GitHawk copied to clipboard

Check boxes don’t toggle correctly

Open drewcassidy opened this issue 5 years ago • 4 comments

Tapping an empty checkbox in an issue adds extra brackets and removes the dash, and is no longer recognized as a checkbox by github

Using the latest GitHawk beta, version 1.28.0 GitHawk Upload by drewcassidy

Bug Report Dump (Auto-generated)
Version 1.28.0 (1554002345)
Device: iPhone 8 (iOS 12.2)
TestFlight: true

drewcassidy avatar May 12 '19 23:05 drewcassidy

Had the same problem before: https://github.com/GitHawkApp/GitHawk/issues/2655

tooomm avatar May 13 '19 07:05 tooomm

@drewcassidy can you share the markdown of the comment, so we can debug it?

BasThomas avatar May 14 '19 14:05 BasThomas

Before:

- [x] Model
- [x] Animate
- [ ] Unwrap
- [ ] Texture
- [ ] Integration
- [ ] QA

After:

- [x] Model
- [x] Animate
[x] ] Unwrap
[x] ] Texture
[x] ] Integration
- [ ] QA

drewcassidy avatar May 14 '19 16:05 drewcassidy

(If this hasn't been solved yet) Are there any other settings you have in GitHawk? e.g., double-tap reactions. I've found that tapping too quickly on checkboxes triggers the double-tap reaction, which might be a bug on its own and interfere with the checkbox.

Unfortunately I haven't been able to reproduce this either.

Just by looking at the code in IssueCommentSectionController#didTapCheckbox, maybe the checkbox.originalMarkdownRange is somehow being miscalculated, causing the replacement below to be invalid?

let edited = originalMarkdownNSString.replacingCharacters(in: range, with: invertedToken)

peterkos avatar Jul 03 '19 06:07 peterkos