Unity

Merge lp://qastaging/~nicovdw/unity/fix-for-733743 into lp://qastaging/unity

  1. fix-for-733743
  2. Merge into trunk
Proposed by Nico van der Walt
Status: Rejected
Rejected by: Gord Allott
Proposed branch: lp://qastaging/~nicovdw/unity/fix-for-733743
Merge into: lp://qastaging/unity
Diff against target: 15 lines (+5/-1)
1 file modified
src/QuicklistMenuItemLabel.cpp (+5/-1)
To merge this branch: bzr merge lp://qastaging/~nicovdw/unity/fix-for-733743
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Disapprove
Gord Allott Pending
Review via email: mp+53336@code.qastaging.launchpad.net

Description of the change

Fixes bug (LP #733743) where items in the Quicklist is not displayed when the text contains an ampersand e.g. 'Science & Engineering'.

To post a comment you must log in.
Revision history for this message
Neil J. Patel (njpatel) wrote :

Sorry nico but I fixed this earlier in the week, but I didn't see this bug so I couldnt' assign it to myself to save you the trouble. Fwiw, the way to do it is to use g_markup_escape_text before sending the string to the quicklist label.

review: Disapprove

Unmerged revisions

950. By Nico van der Walt

Fixes LP: #733743 - Quicklist menus did not display entries that contained an ampersand.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.