diff options
author | Caolán McNamara <caolanm@redhat.com> | 2012-10-22 13:31:38 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2012-10-22 13:35:09 +0100 |
commit | 920807d4491834cd88e545fdb930c9105d58c4cd (patch) | |
tree | b8ea75cb07e03b901413a926f63af67609a70675 /vcl | |
parent | 3c71a0bae6dab8dc53c5f2b014933dcf7ef45024 (diff) |
Resolves: rhbz#868479 / fdo#56281 doubled ~ in German ok/cancel translations
We have unix-specific code adding ~ to OK/Cancel. So don't add ~ if
string already contains those.
Though its an open question if the presence of ~ is a bad thing
for the Windows case. i.e. if we should have tooling to not
allow the OK/Cancel translations to contain ~ in the first place,
of if we should drop the ifdef UNX and do it globally now
Change-Id: I461c6ac9ca574ed188f51472919be82ec582e389
Diffstat (limited to 'vcl')
-rw-r--r-- | vcl/source/control/button.cxx | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/vcl/source/control/button.cxx b/vcl/source/control/button.cxx index 373143b6191d..d2336ecf0133 100644 --- a/vcl/source/control/button.cxx +++ b/vcl/source/control/button.cxx @@ -157,12 +157,15 @@ XubString Button::GetStandardText( StandardButtonType eButton ) sal_uInt32 nResId = aResIdAry[(sal_uInt16)eButton].nResId; aText = ResId(nResId, *pResMgr).toString(); - // Windows (apparently) has some magic auto-accelerator evil around - // ok / cancel so add this only for Unix -#ifdef UNX - if( nResId == SV_BUTTONTEXT_OK || nResId == SV_BUTTONTEXT_CANCEL ) - aText.Insert( rtl::OUString("~"), 0 ); + if (nResId == SV_BUTTONTEXT_OK || nResId == SV_BUTTONTEXT_CANCEL) + { +#ifndef WNT + // Windows (apparently) has some magic auto-accelerator evil around + // ok / cancel so add accelerators only for Unix + if (aText.Search('~') == STRING_NOTFOUND) + aText.Insert(rtl::OUString("~"), 0); #endif + } } else { |