Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs17.4 #7662

Merged
merged 11 commits into from Nov 2, 2017
Merged

Docs17.4 #7662

merged 11 commits into from Nov 2, 2017

Conversation

josephsl
Copy link
Collaborator

Link to issue number:

None

Summary of the issue:

Documentation updates and edits.

Description of how this pull request fixes the issue:

Added Macedonian as a supported language, edits to what's new document and user guide, edited URL's displayed on user guide to point to newer websites (in case of developer guide) and use HTTPS (Add-ons website).

Testing performed:

None

Known issues with pull request:

None

Change log entry:

None

* WDAG: clarified which version of Windows 10 introduced it (Fall Creators Update).
* espeak-ng: shortened commit hash to eight characters.
* Addons -> add-ons.
* Portable copy command line: remove 'support for' to make the entry consistent.
* Braille dispaly key assignment: punctuation change.
…s for developer Guide and extra voices wiki page.
@josephsl
Copy link
Collaborator Author

If no objections are raised, I recommend merging this into master before translatable string freeze. Thanks,


++ Reload plugins ++
This item, once activated, reloads app modules and global plugins without restarting NVDA, which can be useful for developers.

+ Supported Speech Synthesizers +[SupportedSpeechSynths]
This section contains information about the speech synthesizers supported by NVDA.
For an even more extensive list of free and commercial synthesizers that you can purchase and download for use with NVDA, please see the page at [http://www.nvda-project.org/wiki/ExtraVoices http://www.nvda-project.org/wiki/ExtraVoices].
For an even more extensive list of free and commercial synthesizers that you can purchase and download for use with NVDA, please see [extra voices page https://github.com/nvaccess/nvda/wiki/ExtraVoices].
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps "see the extra voices page" instead of "see extra voices page".

Copy link
Collaborator

Choose a reason for hiding this comment

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

agree.

@josephsl
Copy link
Collaborator Author

josephsl commented Oct 15, 2017 via email

Copy link
Collaborator

@derekriemer derekriemer left a comment

Choose a reason for hiding this comment

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

Have you checked for any other references to nvda-project.org, community.*?

- Version information for addons is now reported when being updated. (#5324)
- Added support for new command line parameters to create a portable copy of NVDA. (#6329)
- Support for Microsoft Edge running within Windows Defender Application Guard (#7600)
- Version information for add-ons is now reported when being updated. (#5324)
Copy link
Collaborator

Choose a reason for hiding this comment

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

is -> are?

- Added support for new command line parameters to create a portable copy of NVDA. (#6329)
- Support for Microsoft Edge running within Windows Defender Application Guard (#7600)
- Version information for add-ons is now reported when being updated. (#5324)
- Added new command line parameters to create a portable copy of NVDA. (#6329)
Copy link
Collaborator

Choose a reason for hiding this comment

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

what did you change here?

- Support for Microsoft Edge running within Windows Defender Application Guard (#7600)
- Version information for add-ons is now reported when being updated. (#5324)
- Added new command line parameters to create a portable copy of NVDA. (#6329)
- Support for Microsoft Edge running within Windows Defender Application Guard in Windows 10 Fall Creators Update. (#7600)
Copy link
Collaborator

Choose a reason for hiding this comment

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

(and above)?

- If running on a laptop or tablet, NVDA will now report when a charger is connected/disconnected, and when the screen orientation changes. (#4574, #4612)
- New language: Macedonian.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd say this more flowery. Like
NVDA has now been translated into Macedonian!

Copy link
Collaborator

Choose a reason for hiding this comment

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

not going to make it flowery?

- Web dialogs in Firefox and Chrome web browsers now automatically use browse Mode, unless inside of a web application. (#4493)
- In browse mode, tabbing and moving with quick navigation commands no longer announces jumping out of containers such as lists and tables, which makes navigating more efficient. (#2591)
- In Browse mode for Firefox and Chrome, the name of form field groups are now announced when moving into them with quick navigation or when tabbing. (#3321)
- In browse mode, the quick navigation command for embedded objects (o and shift+o) now includes audio and video elements as well as elements with the aria roles application and dialog. (#7239)
- Espeak-ng has been updated (to commit 01919cd48a566cdf34347784b2e74554b376e900), this resolves some issues with producing release builds. (#7385)
- Espeak-ng has been updated to commit 01919cd4, which resolves some issues with producing release builds. (#7385)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Arguably, () around the commit stuff was more correct since it lets the non-essential info be in parenthesis. However, the way you did this is more compact. If noone else objects, I think this is better. Maybe a comma before to could make it slightly more obvious the commit info is extra, but it really doesn't matter.

@@ -34,7 +35,7 @@ What's New in NVDA
- NVDA no longer becomes noticeably slower when many tabs are open in either Firefox or Chrome web browsers. (#3138)
- Cursor routing for the MDV Lilli Braille display no longer incorrectly moves one braille cell ahead of where it should be. (#7469)
- In Internet Explorer and other MSHTML documents, the HTML5 required attribute is now supported to indicate the required state of a form field. (#7321)
- Braille displays are now updated when typing Arabic characters in a left-aligned WordPad document (#511).
- Braille displays are now updated when typing Arabic characters in a left-aligned WordPad document. (#511)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice catch.


++ Reload plugins ++
This item, once activated, reloads app modules and global plugins without restarting NVDA, which can be useful for developers.

+ Supported Speech Synthesizers +[SupportedSpeechSynths]
This section contains information about the speech synthesizers supported by NVDA.
For an even more extensive list of free and commercial synthesizers that you can purchase and download for use with NVDA, please see the page at [http://www.nvda-project.org/wiki/ExtraVoices http://www.nvda-project.org/wiki/ExtraVoices].
For an even more extensive list of free and commercial synthesizers that you can purchase and download for use with NVDA, please see [extra voices page https://github.com/nvaccess/nvda/wiki/ExtraVoices].
Copy link
Collaborator

Choose a reason for hiding this comment

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

agree.

@josephsl
Copy link
Collaborator Author

josephsl commented Oct 15, 2017 via email

@michaelDCurran
Copy link
Member

where is this up to? translation freeze really needs to be tomorrow if possible.

# Conflicts:
#	user_docs/en/changes.t2t
@josephsl
Copy link
Collaborator Author

josephsl commented Nov 1, 2017 via email

@michaelDCurran
Copy link
Member

michaelDCurran commented Nov 1, 2017 via email

Copy link
Collaborator

@derekriemer derekriemer left a comment

Choose a reason for hiding this comment

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

You updated espeak from master it looks like. Please make sure you don't cause an espeak regression in master before the freeze.

- If running on a laptop or tablet, NVDA will now report when a charger is connected/disconnected, and when the screen orientation changes. (#4574, #4612)
- New language: Macedonian.
Copy link
Collaborator

Choose a reason for hiding this comment

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

not going to make it flowery?

@@ -25,13 +26,13 @@ What's New in NVDA
- In browse mode, tabbing and moving with quick navigation commands no longer announces jumping out of containers such as lists and tables, which makes navigating more efficient. (#2591)
- In Browse mode for Firefox and Chrome, the name of form field groups are now announced when moving into them with quick navigation or when tabbing. (#3321)
- In browse mode, the quick navigation command for embedded objects (o and shift+o) now includes audio and video elements as well as elements with the aria roles application and dialog. (#7239)
- Espeak-ng has been updated to commit 01919cd4, which resolves some issues with producing release builds. (#7385)
- Espeak-ng has been updated (to release 1.49.2), this resolves some issues with producing release builds. (#7385, #7583)
- On the third activation of the 'read status bar' command, it's contents is copied to the clipboard. (#1785)
Copy link
Collaborator

Choose a reason for hiding this comment

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

please fix it's to its

Copy link
Member

@michaelDCurran michaelDCurran left a comment

Choose a reason for hiding this comment

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

include/espeak has been incorrectly changed.

Copy link
Member

@michaelDCurran michaelDCurran left a comment

Choose a reason for hiding this comment

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

I corrected the eSpeak submodule commit back to eSpeak 1.49.2

@michaelDCurran michaelDCurran merged commit 868bbe5 into nvaccess:master Nov 2, 2017
@nvaccessAuto nvaccessAuto added this to the 2017.4 milestone Nov 2, 2017
@josephsl
Copy link
Collaborator Author

josephsl commented Nov 2, 2017 via email

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.

None yet

5 participants