Skip to content

translate excel sheet name reporting message#6866

Merged
feerrenrut merged 2 commits into
masterfrom
i6848-makeCurrentExcelWorksheetTextTranslated
Mar 14, 2017
Merged

translate excel sheet name reporting message#6866
feerrenrut merged 2 commits into
masterfrom
i6848-makeCurrentExcelWorksheetTextTranslated

Conversation

@feerrenrut

Copy link
Copy Markdown
Contributor

fixes #6848

rowAndColumn = self.cellCoordsText
sheet = self.excelWindowObject.ActiveSheet.name
return u"Sheet {0}, {1}".format(sheet, rowAndColumn)
return _(u"Sheet {0}, {1}").format(sheet, rowAndColumn)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You need a translator comment above this explaining what the string is.
E.g.

# Translators: blah blah blah

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oops of course I do. Not sure how I forgot that.

feerrenrut added a commit that referenced this pull request Feb 14, 2017
For issue #6848
Merge branch 'i6848-makeCurrentExcelWorksheetTextTranslated' into next
@feerrenrut feerrenrut merged commit 9880a8c into master Mar 14, 2017
@nvaccessAuto nvaccessAuto added this to the 2017.2 milestone Mar 14, 2017
feerrenrut added a commit that referenced this pull request Mar 14, 2017
For PR #6761 - Web page menu items (menu item checkbox's and radio buttons) can now be activated while in browse mode.  Issue #6735
For PR #6866 - Excel sheet name reporting is now translated. Issue #6848
For PR #6884 - Pressing ESC while the configuration profile "Confirm Deletion" prompt is active now dismisses the dialog. Issue #6851
For PR #6895 - Cell border information can now be reported in Microsoft Excel by using `NVDA+f`. Issue #3044
For PR #6860 - Added support for aria-current attributes. Issue #6358
@feerrenrut feerrenrut deleted the i6848-makeCurrentExcelWorksheetTextTranslated branch September 21, 2017 01:30
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.

Unicode Encode error in Microsoft Excel

3 participants