diff --git a/src/laybasic/laybasic/gsiDeclLayLayoutViewBase.cc b/src/laybasic/laybasic/gsiDeclLayLayoutViewBase.cc index a36fd0468..2c994a69d 100644 --- a/src/laybasic/laybasic/gsiDeclLayLayoutViewBase.cc +++ b/src/laybasic/laybasic/gsiDeclLayLayoutViewBase.cc @@ -1058,7 +1058,7 @@ LAYBASIC_PUBLIC Class decl_LayoutViewBase ("lay", "LayoutVi "\n" "This method has been introduced in version 0.26.2\n" ) + - gsi::method ("select_all", (void (lay::LayoutViewBase::*) ()) &lay::LayoutViewBase::select, + gsi::method ("select_all", (void (lay::LayoutViewBase::*) ()) &lay::LayoutViewBase::select_all, "@brief Selects all objects from the view\n" "\n" "This method has been introduced in version 0.27\n" diff --git a/src/laybasic/laybasic/layEditable.cc b/src/laybasic/laybasic/layEditable.cc index 2476caddd..17cc210d3 100644 --- a/src/laybasic/laybasic/layEditable.cc +++ b/src/laybasic/laybasic/layEditable.cc @@ -369,23 +369,6 @@ Editables::clear_selection () } } -void -Editables::select () -{ - cancel_edits (); - clear_transient_selection (); - clear_previous_selection (); - - for (iterator e = begin (); e != end (); ++e) { - if (m_enabled.find (&*e) != m_enabled.end ()) { - e->select (db::DBox (), lay::Editable::Replace); // select "all" - } - } - - // send a signal to the observers - signal_selection_changed (); -} - void Editables::select (const db::DBox &box, lay::Editable::SelectionMode mode) { diff --git a/src/laybasic/laybasic/layEditable.h b/src/laybasic/laybasic/layEditable.h index 77107a0d7..486223981 100644 --- a/src/laybasic/laybasic/layEditable.h +++ b/src/laybasic/laybasic/layEditable.h @@ -507,11 +507,6 @@ public: */ void clear_previous_selection (); - /** - * @brief Select "all" - */ - void select (); - /** * @brief Select geometrically by a rectangle */ diff --git a/src/laybasic/laybasic/layFinder.cc b/src/laybasic/laybasic/layFinder.cc index 52eac2dad..b874cff6c 100644 --- a/src/laybasic/laybasic/layFinder.cc +++ b/src/laybasic/laybasic/layFinder.cc @@ -324,7 +324,7 @@ ShapeFinder::find (LayoutViewBase *view, const db::DBox ®ion_mu) m_cells_with_context.clear (); lay::TextInfo text_info (view); - mp_text_info = (m_flags & db::ShapeIterator::Texts) != 0 ? &text_info : 0; + mp_text_info = (m_flags & db::ShapeIterator::Texts) != 0 && point_mode () ? &text_info : 0; std::vector lprops; for (lay::LayerPropertiesConstIterator lp = view->begin_layers (); ! lp.at_end (); ++lp) { diff --git a/src/laybasic/laybasic/layLayoutViewBase.cc b/src/laybasic/laybasic/layLayoutViewBase.cc index 108644c5e..9f71719d7 100644 --- a/src/laybasic/laybasic/layLayoutViewBase.cc +++ b/src/laybasic/laybasic/layLayoutViewBase.cc @@ -3753,6 +3753,12 @@ LayoutViewBase::full_box () const return bbox; } +void +LayoutViewBase::select_all () +{ + select (full_box (), lay::Editable::Replace); +} + void LayoutViewBase::zoom_fit () { diff --git a/src/laybasic/laybasic/layLayoutViewBase.h b/src/laybasic/laybasic/layLayoutViewBase.h index c6181b6b9..d70da120e 100644 --- a/src/laybasic/laybasic/layLayoutViewBase.h +++ b/src/laybasic/laybasic/layLayoutViewBase.h @@ -2623,6 +2623,11 @@ public: */ db::DBox full_box () const; + /** + * @brief Selects everything + */ + void select_all (); + /** * @brief Gets called when a menu item is activated */ diff --git a/src/layui/layui/layLayoutViewFunctions.cc b/src/layui/layui/layLayoutViewFunctions.cc index 01a133436..7cbe98685 100644 --- a/src/layui/layui/layLayoutViewFunctions.cc +++ b/src/layui/layui/layLayoutViewFunctions.cc @@ -135,7 +135,7 @@ LayoutViewFunctions::menu_activated (const std::string &symbol) } else if (symbol == "cm_unselect_all") { view ()->select (db::DBox (), lay::Editable::Reset); } else if (symbol == "cm_select_all") { - view ()->select (view ()->full_box (), lay::Editable::Replace); + view ()->select_all (); } else if (symbol == "cm_select_next_item") { view ()->repeat_selection (lay::Editable::Replace); } else if (symbol == "cm_select_next_item_add") {