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

Added a new paragraph to the bug and feature issue templates, explaining where to enter comments #12333

Merged
merged 3 commits into from Apr 30, 2021

Conversation

XLTechie
Copy link
Collaborator

@XLTechie XLTechie commented Apr 23, 2021

Link to issue number:

In response to incidental discussion in #11847

Summary of the issue:

At various times, we've seen issue templates filled out in garbled ways, that make it difficult to navigate or work with the issue description.

In the above issue, for example, the initial description--by a sighted user--had the issue's answers and comments entered directly after the colons on the question lines.
This caused all answers to be headers, along with the questions provoking them, which lead to the issue being hard to navigate other than by simply arrowing through it.

In this case the author is a very experienced technical user, but because he did not know Markdown, and had no idea that he should know Markdown, he answered in the way that seemed most logical.

I've also observed worse examples, where the issue details are written after all the section/question headings, and so on.

Description of how this pull request fixes the issue:

This issue can't be fixed while still using a text based template for issues.
However, it may be possible to mitigate it, by including very basic instructions about where answers should be placed in the template.

This PR adds the following comment, after a blank line to make it stand out, at the bottom of the top HTML issue comment for the bug and feature templates:

Each one of the questions and sections below start with a set of hashmarks (#). Place your answers and information on the blank line below each question.

This language was the best of a few variations I came up with, but I'm sure it could be improved upon. Still, I think it's better than nothing.

We can't make people read the comment paragraphs, but at least providing the definite instruction will reach those who do.

Edit: Subsequently, I added this to the default issue template as well. And while it's unrelated, I added the revised COM Reg. Fix tool question from #12276 to the default template since I was there. If this isn't approved, I'll do a separate PR for that.

Testing strategy:

N/A

Known issues with pull request:

The term I used, "hashmark", is both shorter than "number sign", and I think somewhat more global. Whether it needs a space, or whether one of the other terms would be better, is debatable, and I'm fine with anything that improves clarity.
This Wikipedia article section has a list of terms, and Wikipedia's idea of how popular they are.

Change log entries:

None needed.

Code Review Checklist:

No code changed.

@XLTechie
Copy link
Collaborator Author

cc @britechguy, @Qchristensen

@XLTechie XLTechie marked this pull request as ready for review April 24, 2021 01:54
@XLTechie XLTechie requested a review from a team as a code owner April 24, 2021 01:54
@XLTechie XLTechie requested a review from seanbudd April 24, 2021 01:54
@AppVeyorBot
Copy link

See test results for failed build of commit 3f9f39ca46

@XLTechie
Copy link
Collaborator Author

Chrome TEST i7562 was what failed here, and I have no idea why since this PR doesn't touch anything related.

Full Name:
Robot.chromeTests.i7562
Documentation:
List should not be announced on every line of a ul in a contenteditable
Tags:
browser, chrome, NVDA, smoke test
Start / End / Elapsed:
20210423 21:53:59.686 / 20210423 21:54:08.427 / 00:00:08.741
Status:
FAIL (critical)
Message:
Specific speech did not occur before timeout: NVDA Browser Test Case
See NVDA log for dump of all speech.

…s, explaining that comments should appear BENEATH the lines with hashmarks.
@XLTechie XLTechie force-pushed the issueTemplatesAnswerPlacement branch from a046876 to f6104c6 Compare April 24, 2021 20:11
@CyrilleB79
Copy link
Collaborator

Should also the colon at the end of the section title be removed?
According to #11847 (comment), it was misleading.

@XLTechie
Copy link
Collaborator Author

XLTechie commented Apr 24, 2021 via email

@CyrilleB79
Copy link
Collaborator

I was thinking to remove colons but keep question marks.
Anyway, it is not so important IMO.

I also vote against the comment at the end of the line. It makes things to heavy when editing.
And there is the risk that some people inadvertently remove a '<', '-' or '>' symbol making this comment appear in the final result. We already some bug having this issue with the initial comment.

We should also keep in mind that the vast majority of bug templates are filled correctly, even by people that did not know markdown before (myself included).
So we need to check in this PR that the modifications do not degrade the experience of people that already fill the bug template correctly.

@LeonarddeR
Copy link
Collaborator

If filling these templates is such a problem, we really should consider switching to Github Issue Forms at some point.

@Qchristensen
Copy link
Member

Looks good, but random question: I've never heard the term "hashmarks". I've always seen them referred to as "the hash symbol" or simply "hash".

Personally, I would reword the first part of each change from:
"Each one of the questions and sections below start with a set of hashmarks (#)."
to:
"Each of the questions and sections below start with multiple hash symbols (#)."

(The change to the COM registration fixing tool is fine as is).

@XLTechie
Copy link
Collaborator Author

XLTechie commented Apr 26, 2021 via email

@LeonarddeR
Copy link
Collaborator

LeonarddeR commented Apr 26, 2021 via email

@XLTechie
Copy link
Collaborator Author

XLTechie commented Apr 26, 2021 via email

@XLTechie
Copy link
Collaborator Author

XLTechie commented Apr 26, 2021 via email

Copy link
Contributor

@feerrenrut feerrenrut left a comment

Choose a reason for hiding this comment

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

Thanks @XLTechie

@feerrenrut
Copy link
Contributor

One other thing to make this more clear might be to include a complete raw markdown example on the wiki:
https://github.com/nvaccess/nvda/wiki/Github-issue-template-explanation-and-examples. If you paste it in a comment I can update the page.

@feerrenrut feerrenrut merged commit f392c23 into nvaccess:master Apr 30, 2021
@nvaccessAuto nvaccessAuto added this to the 2021.1 milestone Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants