<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <p>The bug occurs in code that I wrote myself. Only the first column
      in the treeview is editable.</p>
    <p>But it seems that the rest of the LO code never tries to edit
      second or further columns in treeviews. Otherwise the bug would
      have been found long ago.<br>
    </p>
    <div class="moz-cite-prefix">Am 21.01.25 um 10:28 schrieb Michael
      Weghorn:<br>
    </div>
    <blockquote type="cite"
      cite="mid:b3cf093c-290d-4e54-aeb3-a4f8adba8d2e@posteo.de">On
      2025-01-14 21:47, Jan Rheinländer wrote:
      <br>
      <blockquote type="cite">maybe there is a bug in
        <a class="moz-txt-link-freetext" href="SalInstanceTreeView::set_column_editables()">SalInstanceTreeView::set_column_editables()</a>:
        <br>
        <br>
        It calls <a class="moz-txt-link-freetext" href="SvTabListBox::SetTabEditable()">SvTabListBox::SetTabEditable()</a>, which sets the EDITABLE
        flag in mvTabList array of SvLBoxTab.
        <br>
        <br>
        But the parent class SvTreeListBox keeps another array of
        SvLBoxTab called aTabs.
        <br>
        <br>
        When the user presses the mouse button on a cell in a treeview,
        <a class="moz-txt-link-freetext" href="SvImpLBox::MouseButtonDown()">SvImpLBox::MouseButtonDown()</a> is called.
        <br>
        <br>
        It gets the tab <a class="moz-txt-link-freetext" href="SvTreeListBox::GetTab()">SvTreeListBox::GetTab()</a>, which retrieves it from
        the aTabs array.
        <br>
        <br>
        Then it checks the EDITABLE flag with <a class="moz-txt-link-freetext" href="SvLBoxTab::IsEditable()">SvLBoxTab::IsEditable()</a>.
        <br>
        <br>
        But of course the EDITABLE flag has been set in the mvTabList
        array of the subclass SvTabListBox.
        <br>
        <br>
        The net result is that only the first column is editable, since
        that is always set to editable by default (in both tab arrays).
        <br>
        <br>
        Is this a bug? And how to fix it? What is the point of the
        second tab array in the subclass?
        <br>
      </blockquote>
      <br>
      Unfortunately, I don't have definitive answers to this and I'm not
      particularly familiar with that code myself, but it sounds like it
      could be a bug. Do you know a way to trigger this scenario from an
      actual dialog/... or are the above observations so far completely
      based on reading code?
      <br>
      <br>
      Unless anybody else has more insights, looking into code using
      that (and maybe finding a way to trigger and debug into it) might
      possibly help to get a better understanding why that separate tab
      array might be there and how it's used. (And maybe gives more
      clarity on whether it makes actually sense and/or how to resolve
      the potential issue you're mentioning.)
      <br>
    </blockquote>
  </body>
</html>