Skip to content

Report Link Destination partially broken for link titles with semicolons #14667

@XLTechie

Description

@XLTechie

Steps to reproduce:

In a browser such as Firefox, enter the following into the address bar:

data:text/html,<p>This is a test <a href="https://nvaccess.org/">link; to NV Access</a>.</p>

Navigate to the link, and press Ctrl+k (the report link destination key) twice.

Actual behavior:

NVDA reports the title of the Window shown as:

Destination of: link

And the content of the window as:

to NV Access;
https://nvaccess.org/

Expected behavior:

NVDA reports the title of the Window shown as:

Destination of: link; to NV Access;

And the content of the window as:

https://nvaccess.org/

The reason:

The string (message and title) sent to message.html's javascript from ui.browseableMessage, is naively split on a single semicolon. This could have happened before the link destination feature was added, for any ui.browseableMessage with a semicolon in its title.

Noticed while working on #14641.

Fix coming shortly.

NVDA logs, crash dumps and other attachments:

N/A

System configuration

NVDA installed/portable/running from source:

Portable

NVDA version:

2023.1 beta 1

Windows version:

10 Home

Name and version of other software in use when reproducing the issue:

N/A

Other information about your system:

Other questions

Does the issue still occur after restarting your computer?

Not tested.

Have you tried any other versions of NVDA? If so, please report their behaviors.

Also true for alpha and source builds.

If NVDA add-ons are disabled, is your problem still occurring?

N/A

Does the issue still occur after you run the COM Registration Fixing Tool in NVDA's tools menu?

Yes

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions