Merge lp://qastaging/~unity-team/unity/unity.text-entry-im into lp://qastaging/unity

Proposed by Brandon Schaefer
Status: Merged
Approved by: Tim Penhey
Approved revision: no longer in the source branch.
Merged at revision: 2011
Proposed branch: lp://qastaging/~unity-team/unity/unity.text-entry-im
Merge into: lp://qastaging/unity
Diff against target: 377 lines (+27/-251)
3 files modified
plugins/unityshell/src/DashController.cpp (+2/-0)
plugins/unityshell/src/IMTextEntry.cpp (+24/-221)
plugins/unityshell/src/IMTextEntry.h (+1/-30)
To merge this branch: bzr merge lp://qastaging/~unity-team/unity/unity.text-entry-im
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Needs Fixing
Tim Penhey (community) Approve
Jay Taoko (community) Approve
Thomi Richards (community) Approve
Review via email: mp+94305@code.qastaging.launchpad.net

Commit message

IMTextEntry now only handles copy/cut and pasting, everything else is in nux::TextEntry

To post a comment you must log in.
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

I've had a look through this, and it looks good to me.

review: Approve
Revision history for this message
Jay Taoko (jaytaoko) wrote :

Approved!

review: Approve
Revision history for this message
Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity/333/console reported an error when processing this lp:~unity-team/unity/unity.text-entry-im branch.
Not merging it.

Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

-- checking for modules 'compiz;nux-2.0>=2.0.0;libbamf3;dee-1.0;gio-2.0;gio-unix-2.0;dbusmenu-glib-0.4;x11;libstartup-notification-1.0;gthread-2.0;indicator3-0.4;atk;unity-misc>=0.4.0;gconf-2.0;libutouch-geis;gtk+-3.0>=3.1;sigc++-2.0;json-glib-1.0;libnotify;gnome-desktop-3.0;gdu;ibus-1.0;unity>=4.99.0'
-- package 'ibus-1.0' not found

Hmm I thought "libibus-1.0-dev" had been added to unity

Revision history for this message
Tim Penhey (thumper) wrote :

Changes look good, lets try it.

review: Approve
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :
review: Needs Fixing
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Sorry, I must have missed that while deleting everything! Here is the fix again:
https://code.launchpad.net/~brandontschaefer/unity/unity.fix-middle-paste-reg/+merge/94441

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.