Summary: | escaping of amphersand (&) in context menus | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | Kevin Puetz <kde> |
Component: | general | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | RESOLVED FIXED | ||
Severity: | minor | ||
Priority: | NOR | ||
Version: | 1.2.1 | ||
Target Milestone: | --- | ||
Platform: | Debian testing | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Kevin Puetz
2005-03-11 02:10:14 UTC
dropping severity to minor CVS commit by rolandg: escape '&' char in contextmenu entry CCMAIL: amarok-devel@sf.net BUGS: 101276 M +1 -1 playlist.cpp 1.453 --- kdeextragear-1/amarok/src/playlist.cpp #1.452:1.453 @@ -1838,5 +1838,5 @@ Playlist::showContextMenu( QListViewItem popup.insertItem( trackColumn ? i18n("&Iteratively Assign Track Numbers") - : i18n("Write '%1' For Selected Tracks").arg( KStringHandler::rsqueeze( tag, 30 ) ), FILL_DOWN ); + : i18n("Write '%1' For Selected Tracks").arg( KStringHandler::rsqueeze( tag, 30 ).replace( "&", "&&" ) ), FILL_DOWN ); popup.insertItem( SmallIconSet( "editcopy" ), i18n( "&Copy Meta-String" ), 0, 0, CTRL+Key_C, COPY ); With this patch, "&" is set as an accelerator in the situation reported (anyone confirms?). That shouldn't happen, there should be a propper way of escaping that, but it seems && is really the only option. |