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

An issue in the "Report Issue" dialog with a normal sized headline and one character in "Steps" section is too much data to send to github #82856

Closed
jcollum-autodesk opened this issue Oct 18, 2019 · 14 comments
Assignees
Labels
issue-reporter Issue reporter widget issues polish Cleanup and polish issue
Milestone

Comments

@jcollum-autodesk
Copy link

jcollum-autodesk commented Oct 18, 2019

This is the issue I was trying to report: #82855

I took the"Steps" section down to one character and I still got the above warning.

I'm sure there are reasons for this but the dialog doesn't seem very useful if it can't send an issue with system info and a single character in the dialog box. Maybe you need to refactor this UX.

image

@RMacfarlane
Copy link
Contributor

@jcollum-autodesk To open the new issue page on GitHub with data prefilled, we have to send data through the url query. We don't know what the default browser is, and for some browsers, a url that is too long will result in the page not loading. (There's also a bug with GitHubs login page and long urls even for browsers that don't have a restrictive character limit.)

If we posted the data and actually created the issue on click instead, we

  1. need an auth flow for GitHub
  2. should add a way of previewing the issue text
  3. aren't able to let users attach screenshots at that time since there is no public GitHub api for uploading files

@RMacfarlane RMacfarlane added issue-reporter Issue reporter widget issues feature-request Request for new features or functionality labels Oct 18, 2019
@RMacfarlane RMacfarlane added this to the Backlog milestone Oct 18, 2019
@jcollum-autodesk
Copy link
Author

The system/extension information is probably the culprit. Maybe those should have a copy button for that information that shows after the issue is submitted?

@RMacfarlane
Copy link
Contributor

Yeah, that information is very long.

I don't think separating out the step to include them is a good idea, though. That seems like it would just create more burden to the user, since you would have to go back and forth between the browser and VSCode copying information.

@jcollum-autodesk
Copy link
Author

No matter how you slice it I'm gonna have to do this in a 2 step process:

  1. enter a full description of the issue, come back to grab the system information
  2. get all the system information and then enter the actual description of the issue on the github page

The process would be cleaner if it was clear that I won't be able to put in all the information necessary on the first pass.

How about:

  1. I enter an issue
  2. when I click send a dialog comes up that says "your system information has been copied to the clipboard, please paste into the github issue when that page shows up"
  3. it gets sent to github
  4. when that issue page comes up, I paste the system information into a placeholder section of the issue

Alternately the flow could just ask me to write the issue out when I get to github's issue page. \

It looks like no matter what it's going to be a 2 step process.

@RMacfarlane
Copy link
Contributor

I'm a bit confused, as the flow you've described is what should happen currently. Perhaps the wording of the dialog could be made clearer?

@jcollum-autodesk
Copy link
Author

jcollum-autodesk commented Oct 21, 2019

I was 100% expecting to be able to enter the issue and all the system information in that dialog box. I wasn't able to. That was surprising. POLA says that I should be able to put all the necessary info in that dialog, since there are places for all of it.

I'm saying the workflow needs to be clearer -- I definitely have to do this in 2 passes, since even a few characters in the issue description will result in a failure. That's a limitation of github, which is outside of your control. I shouldn't be hitting failures as part of a normal workflow.

@RMacfarlane
Copy link
Contributor

Sorry, I'm still missing something here, I don't understand why 2 passes are needed. If you click the "Yes" button shown in the original screenshot, everything you have typed in the issue and all of the system information should be copied to the clipboard. The browser should be opened to GitHub and you can then paste it. The current message doesn't indicate that the GitHub issue page will still be opened, which is confusing

@jcollum-autodesk
Copy link
Author

jcollum-autodesk commented Oct 21, 2019

everything you have typed in the issue

But I typed one character, what's the use of that? Why have the textbox at all?

If I am unable to actually type my "Steps to Reproduce" in to that dialog, what is the point of having that textbox there? It's confusing/misleading.

everything you have typed in the issue and all of the system information should be copied to the clipboard.

What? There's no message of any kind telling me that.


2 passes:

  1. enter a full description of the issue,
  2. come back to grab the system information then paste it in to the github page

OR

  1. get all the system information, send to github
  2. enter the actual description of the issue on the github page

I think the upshot of this is that the "Steps to Repro" dialog box is pointless since I can't use it and the system information is probably more important.

@RMacfarlane
Copy link
Contributor

What? There's no message of any kind telling me that.

Once again, I think the text of this message needs revising, but this is what I meant:

Screen Shot 2019-10-21 at 11 53 15 AM

The intended flow is that you

  • type the title and complete steps to reproduce in the issue reporter
  • if this + system information is too long, the dialog appears asking to use your clipboard
  • github is opened and you paste everything

Yes, with even a small number of extensions installed the dialog is likely to appear, but it shouldn't prevent you from writing out the complete "Steps to Reproduce" and having that copied into the issue

@jcollum-autodesk
Copy link
Author

OK yeah I see the message you mean now.

I only have like 6 extensions installed. I suspect that's a normal amount. Keeping that in mind I suspect most users will hit this message.

The flow could be improved. If the list of extensions and system information is too large how about just replacing the "Steps" textbox with a message like "Too much data to send to github, your issue will be created but you'll have to enter Steps to reproduce after the issue is created."

That way I know what's going to happen before I hit an error message.

@RMacfarlane
Copy link
Contributor

RMacfarlane commented Oct 21, 2019

@jcollum-autodesk Can you please try pressing the "Yes" button in that dialog to understand the current flow? The dialog is not meant to prevent you from entering steps to reproduce

@jcollum-autodesk
Copy link
Author

I did that. But before that I trimmed off the "steps to reproduce" a few times to try to get it to fit. I was just thrown off by the error and assumed I had to trim down so it would work.

I was being a bad user and just reacting to the first sentence. I think just changing the string would probably help, something like "This message is too large to send to github directly, please press Yes and we will copy it to your clipboard so you can paste it in to the description box on the github issue that we are about to create"

Something like that but less wordy :D

@RMacfarlane
Copy link
Contributor

Yeah the first sentences does make it sound like it has completely failed. Thanks, I like your suggestion!

@jcollum-autodesk
Copy link
Author

Cool thanks for working through it with me

@RMacfarlane RMacfarlane modified the milestones: Backlog, October 2019 Oct 21, 2019
@RMacfarlane RMacfarlane added polish Cleanup and polish issue and removed feature-request Request for new features or functionality labels Oct 21, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue-reporter Issue reporter widget issues polish Cleanup and polish issue
Projects
None yet
Development

No branches or pull requests

2 participants