include/sfx2/devtools/ObjectInspectorTreeHandler.hxx |    5 
 sfx2/source/devtools/ObjectInspectorTreeHandler.cxx  |   21 -
 sfx2/uiconfig/ui/developmenttool.ui                  |  359 +++++++++----------
 3 files changed, 181 insertions(+), 204 deletions(-)

New commits:
commit db35b9086476259fa2c047f2e4dfe7862d026530
Author:     Gülşah Köse <gulsah.k...@collabora.com>
AuthorDate: Tue May 11 23:34:59 2021 +0300
Commit:     Gülşah Köse <gulsah.k...@collabora.com>
CommitDate: Wed May 12 11:19:25 2021 +0200

    tdf#141426 Remove GtkPaned struct unnecessary tabs.
    
    In development tool we need GtkPaned for only properties tab.
    Unnecessary hide() show() calls for the other tabs(Interfaces, Services,
    Methods) causes complex problems about inital height of the second pane
    with gen plugin.
    
    Change-Id: I7523ef0bc425d7fc97ec5bd9e30e874f88e354f7
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115450
    Tested-by: Jenkins
    Reviewed-by: Gülşah Köse <gulsah.k...@collabora.com>

diff --git a/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx 
b/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx
index 0919bbfa3b9a..a9cf5be3dae6 100644
--- a/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx
+++ b/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx
@@ -40,9 +40,6 @@ private:
     // just the current context
     css::uno::Reference<css::uno::XComponentContext> mxContext;
 
-    // should the paned size be reset to default on resize
-    bool mbPanedResetSize;
-
     static void clearObjectInspectorChildren(std::unique_ptr<weld::TreeView>& 
pTreeView,
                                              weld::TreeIter const& rParent);
     static void handleExpanding(std::unique_ptr<weld::TreeView>& pTreeView,
@@ -86,8 +83,6 @@ public:
     DECL_LINK(NotebookEnterPage, const OString&, void);
     DECL_LINK(NotebookLeavePage, const OString&, bool);
 
-    DECL_LINK(PanedSizeChange, const Size&, void);
-
     void introspect(css::uno::Reference<css::uno::XInterface> const& 
xInterface);
 
     void dispose();
diff --git a/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx 
b/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx
index 62fb08229798..de81bd7df432 100644
--- a/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx
+++ b/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx
@@ -929,7 +929,6 @@ ObjectInspectorTreeHandler::ObjectInspectorTreeHandler(
     std::unique_ptr<ObjectInspectorWidgets>& pObjectInspectorWidgets)
     : mpObjectInspectorWidgets(pObjectInspectorWidgets)
     , mxContext(comphelper::getProcessComponentContext())
-    , mbPanedResetSize(true)
 {
     mpObjectInspectorWidgets->mpInterfacesTreeView->connect_expanding(
         LINK(this, ObjectInspectorTreeHandler, ExpandingHandlerInterfaces));
@@ -962,15 +961,12 @@ ObjectInspectorTreeHandler::ObjectInspectorTreeHandler(
     mpObjectInspectorWidgets->mpToolbar->set_item_sensitive("inspect", false);
     mpObjectInspectorWidgets->mpToolbar->set_item_sensitive("back", false);
 
-    mpObjectInspectorWidgets->mpTextView->hide();
-
     mpObjectInspectorWidgets->mpNotebook->connect_leave_page(
         LINK(this, ObjectInspectorTreeHandler, NotebookLeavePage));
     mpObjectInspectorWidgets->mpNotebook->connect_enter_page(
         LINK(this, ObjectInspectorTreeHandler, NotebookEnterPage));
 
-    mpObjectInspectorWidgets->mpPaned->connect_size_allocate(
-        LINK(this, ObjectInspectorTreeHandler, PanedSizeChange));
+    pObjectInspectorWidgets->mpPaned->set_position(160);
 }
 
 void 
ObjectInspectorTreeHandler::handleExpanding(std::unique_ptr<weld::TreeView>& 
pTreeView,
@@ -1085,8 +1081,6 @@ IMPL_LINK(ObjectInspectorTreeHandler, 
ToolbarButtonClicked, const OString&, rSel
 
 IMPL_LINK(ObjectInspectorTreeHandler, NotebookEnterPage, const OString&, 
rPageId, void)
 {
-    mpObjectInspectorWidgets->mpTextView->hide();
-
     uno::Any aAny = maInspectionStack.back();
     if (!aAny.hasValue())
         return;
@@ -1108,12 +1102,10 @@ IMPL_LINK(ObjectInspectorTreeHandler, 
NotebookEnterPage, const OString&, rPageId
     }
     else if (rPageId == "object_inspector_properties_tab")
     {
-        mbPanedResetSize = true;
         mpObjectInspectorWidgets->mpPropertiesTreeView->freeze();
         clearAll(mpObjectInspectorWidgets->mpPropertiesTreeView);
         appendProperties(xInterface);
         mpObjectInspectorWidgets->mpPropertiesTreeView->thaw();
-        mpObjectInspectorWidgets->mpTextView->show();
     }
     else if (rPageId == "object_inspector_methods_tab")
     {
@@ -1153,17 +1145,6 @@ IMPL_LINK(ObjectInspectorTreeHandler, NotebookLeavePage, 
const OString&, rPageId
     return true;
 }
 
-IMPL_LINK(ObjectInspectorTreeHandler, PanedSizeChange, const Size&, rSize, 
void)
-{
-    if (mbPanedResetSize)
-    {
-        // Set position at 90% of the height
-        tools::Long nHeight = rSize.Height();
-        mpObjectInspectorWidgets->mpPaned->set_position(nHeight * 0.9);
-        mbPanedResetSize = false;
-    }
-}
-
 void ObjectInspectorTreeHandler::clearObjectInspectorChildren(
     std::unique_ptr<weld::TreeView>& pTreeView, weld::TreeIter const& rParent)
 {
diff --git a/sfx2/uiconfig/ui/developmenttool.ui 
b/sfx2/uiconfig/ui/developmenttool.ui
index 826cf30a2fd8..58e03b3cf21a 100644
--- a/sfx2/uiconfig/ui/developmenttool.ui
+++ b/sfx2/uiconfig/ui/developmenttool.ui
@@ -95,6 +95,9 @@
                     <property name="model">liststore1</property>
                     <property name="search-column">0</property>
                     <property name="enable-tree-lines">True</property>
+                    <child internal-child="selection">
+                      <object class="GtkTreeSelection"/>
+                    </child>
                     <child>
                       <object class="GtkTreeViewColumn" id="treecolumn">
                         <property name="resizable">True</property>
@@ -289,114 +292,112 @@
               </packing>
             </child>
             <child>
-              <object class="GtkPaned" id="object_inspector_paned">
+              <object class="GtkNotebook" id="object_inspector_notebookbar">
                 <property name="visible">True</property>
                 <property name="can-focus">True</property>
                 <property name="hexpand">True</property>
                 <property name="vexpand">True</property>
-                <property name="orientation">vertical</property>
                 <child>
-                  <object class="GtkNotebook" 
id="object_inspector_notebookbar">
+                  <object class="GtkScrolledWindow">
                     <property name="visible">True</property>
                     <property name="can-focus">True</property>
                     <property name="hexpand">True</property>
                     <property name="vexpand">True</property>
+                    <property name="shadow-type">in</property>
                     <child>
-                      <object class="GtkScrolledWindow">
+                      <object class="GtkTreeView" id="interfaces_treeview_id">
                         <property name="visible">True</property>
                         <property name="can-focus">True</property>
+                        <property name="receives-default">True</property>
                         <property name="hexpand">True</property>
                         <property name="vexpand">True</property>
-                        <property name="shadow-type">in</property>
+                        <property 
name="model">object_inspector_interfaces_liststore</property>
+                        <property name="search-column">0</property>
+                        <property name="enable-tree-lines">True</property>
                         <child>
-                          <object class="GtkTreeView" 
id="interfaces_treeview_id">
-                            <property name="visible">True</property>
-                            <property name="can-focus">True</property>
-                            <property name="receives-default">True</property>
-                            <property name="hexpand">True</property>
-                            <property name="vexpand">True</property>
-                            <property 
name="model">object_inspector_interfaces_liststore</property>
-                            <property name="search-column">0</property>
-                            <property name="enable-tree-lines">True</property>
+                          <object class="GtkTreeViewColumn" 
id="interfaces_treeviewcolumn1">
+                            <property name="resizable">True</property>
+                            <property name="title" translatable="yes" 
context="developmenttool|name">Name</property>
+                            <property name="clickable">True</property>
+                            <property name="sort-indicator">True</property>
+                            <property name="sort-column-id">0</property>
                             <child>
-                              <object class="GtkTreeViewColumn" 
id="interfaces_treeviewcolumn1">
-                                <property name="resizable">True</property>
-                                <property name="title" translatable="yes" 
context="developmenttool|name">Name</property>
-                                <property name="clickable">True</property>
-                                <property name="sort-indicator">True</property>
-                                <property name="sort-column-id">0</property>
-                                <child>
-                                  <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext4"/>
-                                  <attributes>
-                                    <attribute name="text">0</attribute>
-                                  </attributes>
-                                </child>
-                              </object>
+                              <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext4"/>
+                              <attributes>
+                                <attribute name="text">0</attribute>
+                              </attributes>
                             </child>
                           </object>
                         </child>
                       </object>
                     </child>
-                    <child type="tab">
-                      <object class="GtkLabel" 
id="object_inspector_interfaces_tab">
-                        <property name="visible">True</property>
-                        <property name="can-focus">False</property>
-                        <property name="label" translatable="yes" 
context="developmenttool|interfaces">Interfaces</property>
-                      </object>
-                      <packing>
-                        <property name="tab-fill">False</property>
-                      </packing>
-                    </child>
+                  </object>
+                </child>
+                <child type="tab">
+                  <object class="GtkLabel" 
id="object_inspector_interfaces_tab">
+                    <property name="visible">True</property>
+                    <property name="can-focus">False</property>
+                    <property name="label" translatable="yes" 
context="developmenttool|interfaces">Interfaces</property>
+                  </object>
+                  <packing>
+                    <property name="tab-fill">False</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="GtkScrolledWindow">
+                    <property name="visible">True</property>
+                    <property name="can-focus">True</property>
+                    <property name="hexpand">True</property>
+                    <property name="vexpand">True</property>
+                    <property name="shadow-type">in</property>
                     <child>
-                      <object class="GtkScrolledWindow">
+                      <object class="GtkTreeView" id="services_treeview_id">
                         <property name="visible">True</property>
                         <property name="can-focus">True</property>
+                        <property name="receives-default">True</property>
                         <property name="hexpand">True</property>
                         <property name="vexpand">True</property>
-                        <property name="shadow-type">in</property>
+                        <property 
name="model">object_inspector_services_liststore</property>
+                        <property name="search-column">0</property>
+                        <property name="enable-tree-lines">True</property>
                         <child>
-                          <object class="GtkTreeView" 
id="services_treeview_id">
-                            <property name="visible">True</property>
-                            <property name="can-focus">True</property>
-                            <property name="receives-default">True</property>
-                            <property name="hexpand">True</property>
-                            <property name="vexpand">True</property>
-                            <property 
name="model">object_inspector_services_liststore</property>
-                            <property name="search-column">0</property>
-                            <property name="enable-tree-lines">True</property>
+                          <object class="GtkTreeViewColumn" 
id="services_treeviewcolumn1">
+                            <property name="resizable">True</property>
+                            <property name="title" translatable="yes" 
context="developmenttool|name">Name</property>
+                            <property name="clickable">True</property>
+                            <property name="sort-indicator">True</property>
+                            <property name="sort-column-id">0</property>
                             <child>
-                              <object class="GtkTreeViewColumn" 
id="services_treeviewcolumn1">
-                                <property name="resizable">True</property>
-                                <property name="title" translatable="yes" 
context="developmenttool|name">Name</property>
-                                <property name="clickable">True</property>
-                                <property name="sort-indicator">True</property>
-                                <property name="sort-column-id">0</property>
-                                <child>
-                                  <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext7"/>
-                                  <attributes>
-                                    <attribute name="text">0</attribute>
-                                  </attributes>
-                                </child>
-                              </object>
+                              <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext7"/>
+                              <attributes>
+                                <attribute name="text">0</attribute>
+                              </attributes>
                             </child>
                           </object>
                         </child>
                       </object>
-                      <packing>
-                        <property name="position">1</property>
-                      </packing>
-                    </child>
-                    <child type="tab">
-                      <object class="GtkLabel" 
id="object_inspector_services_tab">
-                        <property name="visible">True</property>
-                        <property name="can-focus">False</property>
-                        <property name="label" translatable="yes" 
context="developmenttool|services">Services</property>
-                      </object>
-                      <packing>
-                        <property name="position">1</property>
-                        <property name="tab-fill">False</property>
-                      </packing>
                     </child>
+                  </object>
+                  <packing>
+                    <property name="position">1</property>
+                  </packing>
+                </child>
+                <child type="tab">
+                  <object class="GtkLabel" id="object_inspector_services_tab">
+                    <property name="visible">True</property>
+                    <property name="can-focus">False</property>
+                    <property name="label" translatable="yes" 
context="developmenttool|services">Services</property>
+                  </object>
+                  <packing>
+                    <property name="position">1</property>
+                    <property name="tab-fill">False</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="GtkPaned" id="object_inspector_paned">
+                    <property name="visible">True</property>
+                    <property name="can-focus">True</property>
+                    <property name="orientation">vertical</property>
                     <child>
                       <object class="GtkScrolledWindow">
                         <property name="visible">True</property>
@@ -478,142 +479,142 @@
                         </child>
                       </object>
                       <packing>
-                        <property name="position">2</property>
-                      </packing>
-                    </child>
-                    <child type="tab">
-                      <object class="GtkLabel" 
id="object_inspector_properties_tab">
-                        <property name="visible">True</property>
-                        <property name="can-focus">False</property>
-                        <property name="label" translatable="yes" 
context="developmenttool|properties">Properties</property>
-                      </object>
-                      <packing>
-                        <property name="position">2</property>
-                        <property name="tab-fill">False</property>
+                        <property name="resize">True</property>
+                        <property name="shrink">True</property>
                       </packing>
                     </child>
                     <child>
                       <object class="GtkScrolledWindow">
                         <property name="visible">True</property>
                         <property name="can-focus">True</property>
-                        <property name="hexpand">True</property>
-                        <property name="vexpand">True</property>
                         <property name="shadow-type">in</property>
                         <child>
-                          <object class="GtkTreeView" id="methods_treeview_id">
+                          <object class="GtkTextView" 
id="object_inspector_text_view">
+                            <property name="height-request">100</property>
                             <property name="visible">True</property>
                             <property name="can-focus">True</property>
-                            <property name="receives-default">True</property>
-                            <property name="hexpand">True</property>
-                            <property name="vexpand">True</property>
-                            <property 
name="model">object_inspector_methods_liststore</property>
-                            <property name="search-column">0</property>
-                            <property name="enable-tree-lines">True</property>
-                            <child>
-                              <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn1">
-                                <property name="resizable">True</property>
-                                <property name="title" translatable="yes" 
context="developmenttool|method">Method</property>
-                                <property name="clickable">True</property>
-                                <property name="sort-indicator">True</property>
-                                <property name="sort-column-id">0</property>
-                                <child>
-                                  <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext10"/>
-                                  <attributes>
-                                    <attribute name="text">0</attribute>
-                                  </attributes>
-                                </child>
-                              </object>
-                            </child>
-                            <child>
-                              <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn2">
-                                <property name="resizable">True</property>
-                                <property name="title" translatable="yes" 
context="developmenttool|returntype">Return Type</property>
-                                <property name="clickable">True</property>
-                                <property name="sort-indicator">True</property>
-                                <property name="sort-column-id">1</property>
-                                <child>
-                                  <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext11"/>
-                                  <attributes>
-                                    <attribute name="text">1</attribute>
-                                  </attributes>
-                                </child>
-                              </object>
-                            </child>
-                            <child>
-                              <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn3">
-                                <property name="resizable">True</property>
-                                <property name="title" translatable="yes" 
context="developmenttool|parameters">Parameters</property>
-                                <property name="clickable">True</property>
-                                <property name="sort-indicator">True</property>
-                                <property name="sort-column-id">2</property>
-                                <child>
-                                  <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext12"/>
-                                  <attributes>
-                                    <attribute name="text">2</attribute>
-                                  </attributes>
-                                </child>
-                              </object>
-                            </child>
-                            <child>
-                              <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn4">
-                                <property name="resizable">True</property>
-                                <property name="title" translatable="yes" 
context="developmenttool|implementation_class">Implementation Class</property>
-                                <property name="clickable">True</property>
-                                <property name="sort-indicator">True</property>
-                                <property name="sort-column-id">3</property>
-                                <child>
-                                  <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext13"/>
-                                  <attributes>
-                                    <attribute name="text">3</attribute>
-                                  </attributes>
-                                </child>
-                              </object>
-                            </child>
+                            <property name="editable">False</property>
+                            <property name="wrap-mode">word</property>
+                            <property name="cursor-visible">False</property>
+                            <property name="accepts-tab">False</property>
+                            <property name="monospace">True</property>
                           </object>
                         </child>
                       </object>
                       <packing>
-                        <property name="position">3</property>
-                      </packing>
-                    </child>
-                    <child type="tab">
-                      <object class="GtkLabel" 
id="object_inspector_methods_tab">
-                        <property name="visible">True</property>
-                        <property name="can-focus">False</property>
-                        <property name="label" translatable="yes" 
context="developmenttool|methods">Methods</property>
-                      </object>
-                      <packing>
-                        <property name="position">3</property>
-                        <property name="tab-fill">False</property>
+                        <property name="resize">True</property>
+                        <property name="shrink">True</property>
                       </packing>
                     </child>
                   </object>
                   <packing>
-                    <property name="resize">True</property>
-                    <property name="shrink">True</property>
+                    <property name="position">2</property>
+                  </packing>
+                </child>
+                <child type="tab">
+                  <object class="GtkLabel" 
id="object_inspector_properties_tab">
+                    <property name="visible">True</property>
+                    <property name="can-focus">False</property>
+                    <property name="label" translatable="yes" 
context="developmenttool|properties">Properties</property>
+                  </object>
+                  <packing>
+                    <property name="position">2</property>
+                    <property name="tab-fill">False</property>
                   </packing>
                 </child>
                 <child>
                   <object class="GtkScrolledWindow">
                     <property name="visible">True</property>
                     <property name="can-focus">True</property>
+                    <property name="hexpand">True</property>
+                    <property name="vexpand">True</property>
                     <property name="shadow-type">in</property>
                     <child>
-                      <object class="GtkTextView" 
id="object_inspector_text_view">
-                        <property name="height-request">100</property>
+                      <object class="GtkTreeView" id="methods_treeview_id">
                         <property name="visible">True</property>
                         <property name="can-focus">True</property>
-                        <property name="editable">False</property>
-                        <property name="wrap-mode">word</property>
-                        <property name="cursor-visible">False</property>
-                        <property name="accepts-tab">False</property>
-                        <property name="monospace">True</property>
+                        <property name="receives-default">True</property>
+                        <property name="hexpand">True</property>
+                        <property name="vexpand">True</property>
+                        <property 
name="model">object_inspector_methods_liststore</property>
+                        <property name="search-column">0</property>
+                        <property name="enable-tree-lines">True</property>
+                        <child>
+                          <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn1">
+                            <property name="resizable">True</property>
+                            <property name="title" translatable="yes" 
context="developmenttool|method">Method</property>
+                            <property name="clickable">True</property>
+                            <property name="sort-indicator">True</property>
+                            <property name="sort-column-id">0</property>
+                            <child>
+                              <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext10"/>
+                              <attributes>
+                                <attribute name="text">0</attribute>
+                              </attributes>
+                            </child>
+                          </object>
+                        </child>
+                        <child>
+                          <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn2">
+                            <property name="resizable">True</property>
+                            <property name="title" translatable="yes" 
context="developmenttool|returntype">Return Type</property>
+                            <property name="clickable">True</property>
+                            <property name="sort-indicator">True</property>
+                            <property name="sort-column-id">1</property>
+                            <child>
+                              <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext11"/>
+                              <attributes>
+                                <attribute name="text">1</attribute>
+                              </attributes>
+                            </child>
+                          </object>
+                        </child>
+                        <child>
+                          <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn3">
+                            <property name="resizable">True</property>
+                            <property name="title" translatable="yes" 
context="developmenttool|parameters">Parameters</property>
+                            <property name="clickable">True</property>
+                            <property name="sort-indicator">True</property>
+                            <property name="sort-column-id">2</property>
+                            <child>
+                              <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext12"/>
+                              <attributes>
+                                <attribute name="text">2</attribute>
+                              </attributes>
+                            </child>
+                          </object>
+                        </child>
+                        <child>
+                          <object class="GtkTreeViewColumn" 
id="methods_treeviewcolumn4">
+                            <property name="resizable">True</property>
+                            <property name="title" translatable="yes" 
context="developmenttool|implementation_class">Implementation Class</property>
+                            <property name="clickable">True</property>
+                            <property name="sort-indicator">True</property>
+                            <property name="sort-column-id">3</property>
+                            <child>
+                              <object class="GtkCellRendererText" 
id="obj_insp_cellrenderertext13"/>
+                              <attributes>
+                                <attribute name="text">3</attribute>
+                              </attributes>
+                            </child>
+                          </object>
+                        </child>
                       </object>
                     </child>
                   </object>
                   <packing>
-                    <property name="resize">True</property>
-                    <property name="shrink">True</property>
+                    <property name="position">3</property>
+                  </packing>
+                </child>
+                <child type="tab">
+                  <object class="GtkLabel" id="object_inspector_methods_tab">
+                    <property name="visible">True</property>
+                    <property name="can-focus">False</property>
+                    <property name="label" translatable="yes" 
context="developmenttool|methods">Methods</property>
+                  </object>
+                  <packing>
+                    <property name="position">3</property>
+                    <property name="tab-fill">False</property>
                   </packing>
                 </child>
               </object>
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to