D8695: Fix ampersand characters not showing in tab titles

Adriaan de Groot noreply at phabricator.kde.org
Tue Nov 7 15:22:41 UTC 2017


adridg added inline comments.

INLINE COMMENTS

> Workspace.cpp:142
> +    // Title ampersands must be doubled or will not be displayed and used as QT shortcut
> +    insertTab(-1, sheet, dlg.sheetTitle().replace( "&", "&&" ) );
>      mSheetList.append( sheet );

Are you sure this won't affect tabs that *do* have shortcuts, and need them? How about after a save/restart cycle, are the ampersands re-doubled?

REPOSITORY
  R106 KSysguard

REVISION DETAIL
  https://phabricator.kde.org/D8695

To: corentinr
Cc: adridg, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171107/523196d7/attachment.html>


More information about the Plasma-devel mailing list