Skip to content

Conversation

@seanbudd
Copy link
Member

@seanbudd seanbudd commented Aug 11, 2025

Link to issue number:

Raised in #15320 (comment)

Summary of the issue:

There's no easy way for an NVDA user to copy the version information from the about dialog

Description of user facing changes:

A button has been added to the about dialog to copy the version information.
example source-master-ed86524 (2025.3.0.0)

Description of developer facing changes:

none

Description of development approach:

Added a button the message dialog

Testing strategy:

Tested by opening the about dialog and ensuring the version number is copied

Known issues with pull request:

none

Code Review Checklist:

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

@coderabbitai summary

Copilot AI review requested due to automatic review settings August 11, 2025 02:47
@seanbudd seanbudd requested a review from a team as a code owner August 11, 2025 02:47
@seanbudd seanbudd requested a review from SaschaCowley August 11, 2025 02:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a "Copy version number" button to the NVDA about dialog, making it easier for users to copy version information for support purposes. The button copies the version string in the format source-master-ed86524 (2025.3.0.0).

Key changes:

  • Added a copy button to the about dialog using the existing MessageDialog framework
  • Implemented a callback function to copy version information to clipboard
  • Updated typing annotations to use newer Python syntax

@seanbudd seanbudd mentioned this pull request Aug 11, 2025
6 tasks
Co-authored-by: Leonard de Ruijter <3049216+LeonarddeR@users.noreply.github.com>
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Aug 12, 2025
@seanbudd seanbudd added this to the 2026.1 milestone Aug 13, 2025
@SaschaCowley SaschaCowley enabled auto-merge (squash) August 15, 2025 04:03
@SaschaCowley SaschaCowley merged commit f8f4a31 into master Aug 15, 2025
29 of 30 checks passed
@SaschaCowley SaschaCowley deleted the addCopyVersionBtn branch August 15, 2025 04:36
OzancanKaratas pushed a commit to OzancanKaratas/nvda that referenced this pull request Aug 17, 2025
Raised in
nvaccess#15320 (comment)

Summary of the issue:
There's no easy way for an NVDA user to copy the version information
from the about dialog

Description of user facing changes:
A button has been added to the about dialog to copy the version
information.
example `source-master-ed86524 (2025.3.0.0)`

Description of developer facing changes:
none

Description of development approach:
Added a button the message dialog

Testing strategy:
Tested by opening the about dialog and ensuring the version number is
copied

Known issues with pull request:
none
OzancanKaratas pushed a commit to OzancanKaratas/nvda that referenced this pull request Aug 18, 2025
Raised in
nvaccess#15320 (comment)

Summary of the issue:
There's no easy way for an NVDA user to copy the version information
from the about dialog

Description of user facing changes:
A button has been added to the about dialog to copy the version
information.
example `source-master-ed86524 (2025.3.0.0)`

Description of developer facing changes:
none

Description of development approach:
Added a button the message dialog

Testing strategy:
Tested by opening the about dialog and ensuring the version number is
copied

Known issues with pull request:
none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants