Skip to content

Remove audio logo from NVDA launcher#17507

Merged
SaschaCowley merged 4 commits into
masterfrom
launcherSound
Dec 13, 2024
Merged

Remove audio logo from NVDA launcher#17507
SaschaCowley merged 4 commits into
masterfrom
launcherSound

Conversation

@SaschaCowley

@SaschaCowley SaschaCowley commented Dec 12, 2024

Copy link
Copy Markdown
Member

Link to issue number:

Closes #14068
Closes #15110

Summary of the issue:

The NVDA launcher currently plays an audio logo when launched.
This creates a number of problems:

Description of user facing changes

The NVDA launcher is now silent until the temporary copy of NVDA it includes is started.

Description of development approach

Removed all code related to playing the audio logo from launcher/launcher.nsis.

Testing strategy:

Built the launcher and ran it from explorer.
Also ran NVDA from source patched to check for updates, and swapped the downloaded launcher for the one without the sound logo.

Known issues with pull request:

There is now an appreciable period of silence between NVDA quitting to perform an update, and the temporary NVDA speaking to read out the update progress.

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

@SaschaCowley SaschaCowley marked this pull request as ready for review December 12, 2024 03:03
@SaschaCowley SaschaCowley requested a review from a team as a code owner December 12, 2024 03:03
@cary-rowen

Copy link
Copy Markdown
Contributor

There is now an appreciable period of silence between NVDA quitting to perform an update, and the temporary NVDA speaking to read out the update progress.

Can we improve this? Is it possible to use a shorter sound? In order to let users know that the update has started, rather than being stuck for some reason.

Comment thread launcher/nvdaLauncher.nsi
@SaschaCowley

Copy link
Copy Markdown
Member Author

@cary-rowen

There is now an appreciable period of silence between NVDA quitting to perform an update, and the temporary NVDA speaking to read out the update progress.

Can we improve this? Is it possible to use a shorter sound? In order to let users know that the update has started, rather than being stuck for some reason.

Unfortunately PlaySound and friends are the only easy ways of natively playing a sound file that I'm aware of, and they all use winmm. As we're running under NSIS, our options here are rather limited.

@dpy013

dpy013 commented Dec 13, 2024

Copy link
Copy Markdown
Contributor

Can we take a look at Inno Setup?

@seanbudd

Copy link
Copy Markdown
Member

@dpy013 - we do not intend to add back a launcher sound or migrate from NSIS for the time being. I'd encourage dropping the concern

# Conflicts:
#	user_docs/en/changes.md
@SaschaCowley SaschaCowley merged commit 82fefa5 into master Dec 13, 2024
@SaschaCowley SaschaCowley deleted the launcherSound branch December 13, 2024 03:36
@github-actions github-actions Bot added this to the 2025.1 milestone Dec 13, 2024
seanbudd added a commit that referenced this pull request Dec 18, 2024
…uide (#17519)

Summary of the issue:

In #17507, the audio logo played while the NVDA launcher is loading has been removed. However, the quick start guide still makes a reference to the music playing before a temporary copy of NVDA loads.

Description of user facing changes

The quick start guide no longer includes misleading information. Instead, it has been replaced with a note that it may take a few seconds for the temporary copy to load.


---------

Co-authored-by: Sean Budd <seanbudd123@gmail.com>
@retrochiptuner

Copy link
Copy Markdown

Hello! I agree that this is not a good alternative. In any case with a shorter start sound. Anyway on my computer the installer of the last 2026.1 Beta 1 and the last alphas also take time to load the temporary copy. My computer is a ASUS X541UAK with an Intel (R) Core (TM) I3-6006U cpu @ 2.00GHZ, 1992 MHz, 2 main processors, 4 logical processors.

@nvaccess nvaccess locked and limited conversation to collaborators Apr 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to silence NVDA installer startup sound when updating NVDA (request from user) Use shorter startup sound for NVDA launcher

5 participants