From 931aa4a4032c99997058e33df72cd2a31e8e5eba Mon Sep 17 00:00:00 2001 From: Tobias Ellinghaus Date: Tue, 28 May 2019 08:48:23 +0200 Subject: Fix align-to of text objects Inkscape used to align text baselines relative to the wrong object when using "First selected" or "Last selected". Closes !699 --- src/ui/dialog/align-and-distribute.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/ui') diff --git a/src/ui/dialog/align-and-distribute.cpp b/src/ui/dialog/align-and-distribute.cpp index c112f965d..a714b96f6 100644 --- a/src/ui/dialog/align-and-distribute.cpp +++ b/src/ui/dialog/align-and-distribute.cpp @@ -135,10 +135,10 @@ void ActionAlign::do_action(SPDesktop *desktop, int index) switch (AlignTarget(prefs->getInt("/dialogs/align/align-to", 6))) { case LAST: - focus = SP_ITEM(*--(selected.end())); + focus = SP_ITEM(selected.back()); break; case FIRST: - focus = SP_ITEM(*selected.begin()); + focus = SP_ITEM(selected.front()); break; case BIGGEST: focus = selection->largestItem(horiz); @@ -844,10 +844,10 @@ private : switch (AlignTarget(prefs->getInt("/dialogs/align/align-to", 6))) { case LAST: - focus = SP_ITEM(*selected.begin()); + focus = SP_ITEM(selected.back()); break; case FIRST: - focus = SP_ITEM(*--(selected.end())); + focus = SP_ITEM(selected.front()); break; case BIGGEST: focus = selection->largestItem(Selection::AREA); @@ -867,7 +867,7 @@ private : default: g_assert_not_reached (); break; - }; + }; if(focus) { if (SP_IS_TEXT (focus) || SP_IS_FLOWTEXT (focus)) { -- cgit v1.2.3