Skip to content

Fix up of PR 9843: Once again report when copying text in browse mode in MS Word.#11848

Merged
michaelDCurran merged 1 commit into
nvaccess:masterfrom
lukaszgo1:I11839
Nov 22, 2020
Merged

Fix up of PR 9843: Once again report when copying text in browse mode in MS Word.#11848
michaelDCurran merged 1 commit into
nvaccess:masterfrom
lukaszgo1:I11839

Conversation

@lukaszgo1

Copy link
Copy Markdown
Contributor

Link to issue number:

Fixes #11839

Summary of the issue:

PR #9843 changed signature of the copyToClipboard method on textInfos but this change has not been reflected in WordDocumentTextInfo class.

Description of how this pull request fixes the issue:

Signature of the copyToClipboard for MS Word now matches one from the base class.

Testing performed:

Ensured that text can once again be reported when copying it in MS Word when in browse mode.

Known issues with pull request:

  • MS Word method which we're using to copy text does not allow to check if the copy actually succeeded or not. Because of this the success message is reported alwayss if there is a selection in a document.

Change log entry:

None needed this is not in a release

@CyrilleB79

Copy link
Copy Markdown
Contributor

Is #11851 trying to fix the same issue as this PR?

@JulienCochuyt

Copy link
Copy Markdown
Contributor

@CyrilleB79 wrote:

Is #11851 trying to fix the same issue as this PR?

Indeed, my bad, I did not notice @lukaszgo1 already proposed this fix before posting.
Closing #11851 in favor of this one.
Thanks.

@CyrilleB79

Copy link
Copy Markdown
Contributor

Tested. It is working on my side.

@michaelDCurran michaelDCurran merged commit 15ab401 into nvaccess:master Nov 22, 2020
@nvaccessAuto nvaccessAuto added this to the 2020.4 milestone Nov 22, 2020
@lukaszgo1 lukaszgo1 deleted the I11839 branch November 22, 2020 22:40
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.

Unable to copy text from MS Word virtual buffer

6 participants