Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Comments render white space #1339

Closed
BasThomas opened this issue Jan 4, 2018 · 15 comments
Closed

Comments render white space #1339

BasThomas opened this issue Jan 4, 2018 · 15 comments
Labels
🐛 bug Unintended behaviour within the app
Milestone

Comments

@BasThomas
Copy link
Collaborator

GitHawk Upload by BasThomas

Happens on both iPad and iPhone. Have a feeling it’s been happening more the last few days? Also happened on previous version though.

Bug Report Dump (Auto-generated)
Version 1.17.0 (1514903367)
Device: iPad7,3 (iOS 11.2.1)
TestFlight: true
@BasThomas
Copy link
Collaborator Author

Issue in question: Moya/Moya#1523

Sent with GitHawk

@BasThomas BasThomas added the 🐛 bug Unintended behaviour within the app label Jan 4, 2018
@rnystrom
Copy link
Member

rnystrom commented Jan 5, 2018

Hmm @BasThomas can you do some debugging here? I can't repro.

@SD10
Copy link
Member

SD10 commented Jan 9, 2018

The spacing is even larger in the latest TestFlight: 1.17.0

I think it’s also making scrolling extremely choppy.

GitHawk Upload by SD10

Sent with GitHawk

@BasThomas
Copy link
Collaborator Author

Haven’t seen it lately - can you tap with three fingers to dig a bit with FLEX?

Sent with GitHawk

@SD10
Copy link
Member

SD10 commented Jan 9, 2018

Sorry @BasThomas I don’t know what you mean.

It’s difficult to reproduce but once it happens, any comments have an excessive amount of spacing on any issue throughout the app. Once I kill GitHawk and reopen it’s gone.

It has happened by viewing an issue on a fresh app start, expanding a comment, posting a comment. Can’t single it down to one specific action and it’s inconsistent.

Sent with GitHawk

@BasThomas
Copy link
Collaborator Author

In Testflight, you can tap with three fingers anywhere on the screen to have additional tools (see image), which enable you to debug the views, network requests and more.

GitHawk Upload by BasThomas

Sent with GitHawk

@SD10
Copy link
Member

SD10 commented Jan 9, 2018

Sorry @BasThomas, noob moment. Thanks for getting me up to speed!

Sent with GitHawk

@BasThomas
Copy link
Collaborator Author

@SD10 have you been seeing it in the last few days / been able to debug with FLEX?

@BasThomas BasThomas removed their assignment Jan 13, 2018
@SD10
Copy link
Member

SD10 commented Jan 14, 2018

I haven't seen any but I've had very limited GitHub activity in the past few days. We could close and re-open if I see it again

@rnystrom
Copy link
Member

I think I saw this today caused by html comments and no actual content. Will investigate.

Sent with GitHawk

@Sherlouk
Copy link
Member

SwifterSwift/SwifterSwift#364

Not sure if the same bug, but the root comment in here has a bunch of HTML comments which render as massive blocks of white space in app

@SD10
Copy link
Member

SD10 commented Jan 17, 2018

Steps to reproduce:

  1. Open any issue / PR
  2. Immediately stop view from scrolling to bottom
  3. White space appears on all comments made through GitHawk and other HTML blocks

Sent with GitHawk

@rnystrom
Copy link
Member

@SD10 hmmm I can’t seem to repro that. The html comments are definitely a thing though. I’m going to fix those for 1.17

Sent with GitHawk

@rnystrom
Copy link
Member

PR fixes this big time, but still want to strip commented HTML out. For example, that Moya issue is totally empty.

@rnystrom rnystrom modified the milestones: 1.17.0, 1.18.0 Jan 20, 2018
@rnystrom
Copy link
Member

Seems to be fixed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐛 bug Unintended behaviour within the app
Projects
None yet
Development

No branches or pull requests

4 participants