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

Loading long strings while debugging is clunky #5109

Closed
vsccarl opened this issue Apr 8, 2016 · 4 comments
Closed

Loading long strings while debugging is clunky #5109

vsccarl opened this issue Apr 8, 2016 · 4 comments
Assignees
Milestone

Comments

@vsccarl
Copy link

vsccarl commented Apr 8, 2016

  • VSCode Version: 0.10.12-alpha
  • OS Version: Windows 10

Steps to Reproduce:

  1. Following String truncation limit lifted to 10000 #4952
  2. Create a string with at least 10,000 characters
  3. Start debugging
  4. Hover over the variable in the debug window

longstringdebugging

There is briefly a white box filled with nothing even though it takes a bit of time to load anyway. It should probably not show the box until the text is ready to be shown.

@vsccarl vsccarl added the v-test label Apr 8, 2016
@weinand weinand assigned isidorn and unassigned weinand Apr 11, 2016
@isidorn isidorn added this to the April 2016 milestone Apr 11, 2016
@isidorn
Copy link
Contributor

isidorn commented Apr 12, 2016

This is just the way that electron renders hover on windows since this is the native hover, not our custom VSCode one.
This also works nicely on Mac, and probably on linux.

Closing as can not fix, since this is electron behavior

@isidorn isidorn closed this as completed Apr 12, 2016
@isidorn
Copy link
Contributor

isidorn commented Apr 28, 2016

We also introduced a 5000 character limit in the hover, so I believe this should behave better now

@isidorn isidorn assigned vsccarl and unassigned isidorn Apr 28, 2016
@vsccarl
Copy link
Author

vsccarl commented Apr 29, 2016

The loading time is better but now sometimes it seems to flash or load multiple times while the cursor is still hovered over it (causing the flashing). longstringdebuggingupdate

@vsccarl vsccarl reopened this Apr 29, 2016
@vsccarl vsccarl assigned isidorn and unassigned vsccarl Apr 29, 2016
@isidorn isidorn modified the milestones: May 2016, April 2016 Apr 29, 2016
@isidorn
Copy link
Contributor

isidorn commented May 4, 2016

We have introduced a limit of 1024 which should make this experience nicer now

@isidorn isidorn closed this as completed May 4, 2016
@isidorn isidorn assigned vsccarl and unassigned isidorn May 4, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants