dwaynebailey / translate

60%

master: 66%

LAST BUILD ON BRANCH bugfix/accesskey_case

branch: bugfix/accesskey_case

CHANGE BRANCH

x

Reset

• bugfix/accesskey_case
• Repentinus-patch-2
• appveyor
• bugfix/bug115_reopened
• bugfix/properties_slash_u
• cleanup/accesskey_tests
• cleanup/autopep8
• cleanup/class_object
• cleanup/deprecated
• cleanup/extraneous_docs
• cleanup/pep257
• cleanup/pep8
• cleanup/release-docs
• docs/README
• docs/deprecation
• docs/mock
• docs/update-sphinx
• docs/windows_install
• feature/bugzilla-migration-fixes
• feature/cli_tests
• feature/ios-checkerstyle
• feature/optrecurse_argparse
• feature/pocount_argparse
• feature/py36
• feature/py3_again
• feature/pypy
• feature/pytest3
• feature/tbx2po
• feature/testing-py3
• feature/usecpo1
• fix/accesskey_space
• fix/bytesio_no_seek
• fix/dtd_invalid_amps
• fix/json_backconversion
• fix/json_ordering
• fix/lang-maxlength
• fix/lang_country_common_name
• fix/lang_identical_translated
• fix/linting_deps
• fix/missing_subtitle_support
• fix/mozlang_trailing_newlines
• fix/multiline_untranslated
• fix/pep8
• fix/po2ts
• fix/po2txt_obsolete
• fix/po_end_slash
• fix/puncspacing-check-rlm
• fix/pydocstyle
• fix/readme_buttons
• fix/threshold_intger_error
• fix/unicode_special_chars
• fix/xliff-states
• fixes/pythonbraces-on-LO
• format/lang_ws_fixes
• khaledhosny/feature/dtd_entities
• l20n-db
• lang-better-ok
• master
• nijel-poencoding
• pep8/E265
• py3/non-py3-deps
• requirements/Babel-2.4.0
• requirements/Sphinx-1.5.4
• requirements/Sphinx-1.5.5
• requirements/Sphinx-1.6.1
• requirements/chardet-3.0.2
• requirements/pytest-3.0.5
• requirements/sphinx-1.4.9
• requirements/sphinx-1.5.3
• requirements/vobject-0.9.4
• test/expand_cli_testing
• test/pr3526-cpo
• tests/appveyor
• travis/cpo1_testing
• travis/meta_docs_and_cache
• travis/pip7
• travis/use-containers
• travis/usecpo2

Build # 287

Build Type

push

travis-ci

Committed by dwaynebailey

Commit Message

accesskeys: combine using correct case

We always thought, or claimed that we combined accesskeys based on
prefering the case before using an alternate case.

It seems that our code actually prefered uppercase and then would try
lowercase.  Thus this string:

"Close Other Tabs" combined with an accesskey of "o" would result in
"Close &Other Tabs" because capital O would win even though lowercase O
was present.  The correct combineation should have been "Cl&ose Other
Tabs", preferring the correct case before the alternate case.

Run Details

13217 of 21895 relevant lines covered (60.37%)

1.81 hits per line