Merge lp:~alan-griffiths/miral/fix-1632325 into lp:miral

Proposed by Alan Griffiths
Status: Merged
Approved by: Brandon Schaefer
Approved revision: 404
Merged at revision: 404
Proposed branch: lp:~alan-griffiths/miral/fix-1632325
Merge into: lp:miral
Diff against target: 20 lines (+10/-0)
1 file modified
miral/window_specification.cpp (+10/-0)
To merge this branch: bzr merge lp:~alan-griffiths/miral/fix-1632325
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Gerry Boland (community) Approve
Review via email: mp+308127@code.launchpad.net

Commit message

Prior to Mir-0.25 mir_connection_create_spec_for_tooltip() doesn't set an edge attachment preference. So give it one.

To post a comment you must log in.
Revision history for this message
Gerry Boland (gerboland) wrote :

LGTM

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

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'miral/window_specification.cpp'
--- miral/window_specification.cpp 2016-09-30 16:29:43 +0000
+++ miral/window_specification.cpp 2016-10-11 14:18:52 +0000
@@ -250,6 +250,16 @@
250 aux_rect_placement_offset = Displacement{params.aux_rect_placement_offset_x.value(), params.aux_rect_placement_offset_y.value()};250 aux_rect_placement_offset = Displacement{params.aux_rect_placement_offset_x.value(), params.aux_rect_placement_offset_y.value()};
251#endif251#endif
252252
253#if MIR_SERVER_VERSION < MIR_VERSION_NUMBER(0, 25, 0)
254 // mir_connection_create_spec_for_tooltip() didn't set an edge_attachment preference
255 if (aux_rect.is_set() && !params.edge_attachment.is_set())
256 {
257 window_placement_gravity = mir_placement_gravity_northwest;
258 aux_rect_placement_gravity = mir_placement_gravity_northeast;
259 placement_hints = mir_placement_hints_flip_any;
260 }
261#endif
262
253 if (params.edge_attachment.is_set() && !placement_hints.is_set())263 if (params.edge_attachment.is_set() && !placement_hints.is_set())
254 {264 {
255 switch (params.edge_attachment.value())265 switch (params.edge_attachment.value())

Subscribers

People subscribed via source and target branches