nvda icon indicating copy to clipboard operation
nvda copied to clipboard

Use GitHub forms for issues

Open seanbudd opened this issue 10 months ago • 6 comments

Link to issue number:

None

Summary of the issue:

GitHub issue forms provide an easier and more clear way for users to report issues on the NVDA project.

Currently, when filling out an issue form, inline comments and an external guide is required to understand how to fill out an issue.

Description of user facing changes

When reporting an issue to NVDA, users can use the simpler GitHub forms interface. Information from inline comments and the issue guide have been merged into this form, so that it is easier to understand the requirements.

Additionally, a new "custom issue" has been created, as we disable opening blank issues. This should be used in the rare circumstance when the bug report or feature request template is not appropriate

Description of development approach

N/A

Testing strategy:

These changes have been pushed to a test repository

The new issue interface can be tested here: https://github.com/nvaccess/nvda-issue-form-test/issues/new/choose

Example issues filled out using the form are as follows:

  • https://github.com/nvaccess/nvda-issue-form-test/issues/1
  • https://github.com/nvaccess/nvda-issue-form-test/issues/2
  • https://github.com/nvaccess/nvda-issue-form-test/issues/3

Known issues with pull request:

This PR is blocked by accessibility issues outlined in: https://github.com/orgs/community/discussions/61696. The current information for the bug report may be overly verbose. However, it is rare that an issue reporter reads our external guide for reporting issues, and as such, inline information is preferred.

Change log entries:

N/A

Code Review Checklist:

  • [x] Pull Request description:
    • description is up to date
    • change log entries
  • [x] Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • [x] API is compatible with existing add-ons.
  • [x] Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • [x] UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • [x] Security precautions taken.

seanbudd avatar Aug 22 '23 04:08 seanbudd

Note: re-opened take of https://github.com/nvaccess/nvda/pull/15182, refer to comments and reviews there as well

seanbudd avatar Aug 22 '23 04:08 seanbudd

  • PASS: Translation comments check.
  • PASS: Unit tests.
  • PASS: Lint check.
  • FAIL: System tests (tags: installer NVDA). See test results for more information.
  • Build (for testing PR): https://ci.appveyor.com/api/buildjobs/fl6ul678drpayfw7/artifacts/output/nvda_snapshot_pr15320-28929,672eacd9.exe
  • CI timing (mins): INIT 0.0, INSTALL_START 0.8, INSTALL_END 0.9, BUILD_START 0.0, BUILD_END 25.3, TESTSETUP_START 0.0, TESTSETUP_END 0.3, TEST_START 0.0, TEST_END 22.4, FINISH_END 0.1

See test results for failed build of commit 672eacd963

AppVeyorBot avatar Aug 22 '23 05:08 AppVeyorBot

  • PASS: Translation comments check.
  • PASS: Unit tests.
  • PASS: Lint check.
  • FAIL: System tests (tags: installer NVDA). See test results for more information.
  • Build (for testing PR): https://ci.appveyor.com/api/buildjobs/oiy015l27dm3vt37/artifacts/output/nvda_snapshot_pr15320-29047,f7fb95af.exe
  • CI timing (mins): INIT 0.0, INSTALL_START 0.8, INSTALL_END 0.8, BUILD_START 0.0, BUILD_END 23.7, TESTSETUP_START 0.0, TESTSETUP_END 0.3, TEST_START 0.0, TEST_END 22.1, FINISH_END 0.2

See test results for failed build of commit f7fb95afcb

AppVeyorBot avatar Sep 01 '23 05:09 AppVeyorBot

@seanbudd could we wait for our call next week before merging this? I have some proposals / ideas that we could discuss there.

Adriani90 avatar Apr 24 '24 08:04 Adriani90

@Adriani90 - this PR is still blocked by work from GitHub

seanbudd avatar Apr 24 '24 08:04 seanbudd

I've tested the forms and yeah, I think the most annoying part is that the labels of the fields are dissociated from the fields when the focus lands on them. I think this is still a showstopper. But also the edit fields seem to be smaller in height, at least that's what I read in some forums. This might be very annoying for people with some vision ability.

@seanbudd did you explore the possibility of providing accessible landing pages for the issue templates with the ability of creating githug issues from those pages?

Or maybe there is an alternative tool for customizing the github issue interface? I found this one but it seems inactive: https://github.com/wei/new-issue

Adriani90 avatar May 13 '24 19:05 Adriani90