<html>
    <head>
      <base href="https://bugs.documentfoundation.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:kelemeng@ubuntu.com" title="Gabor Kelemen <kelemeng@ubuntu.com>"> <span class="fn">Gabor Kelemen</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - FILEOPEN XLS LibreOffice doesn't indicate the 256 column limitation of the format when editing XLS files"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=123100">bug 123100</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">CC</td>
           <td>
                
           </td>
           <td>libreoffice-ux-advise@lists.freedesktop.org
           </td>
         </tr>

         <tr>
           <td style="text-align:right;">Keywords</td>
           <td>
                
           </td>
           <td>needsUXEval
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - FILEOPEN XLS LibreOffice doesn't indicate the 256 column limitation of the format when editing XLS files"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=123100#c9">Comment # 9</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - FILEOPEN XLS LibreOffice doesn't indicate the 256 column limitation of the format when editing XLS files"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=123100">bug 123100</a>
              from <span class="vcard"><a class="email" href="mailto:kelemeng@ubuntu.com" title="Gabor Kelemen <kelemeng@ubuntu.com>"> <span class="fn">Gabor Kelemen</span></a>
</span></b>
        <pre>(In reply to Xisco FaulĂ­ from <a href="show_bug.cgi?id=123100#c8">comment #8</a>)
<span class="quote">> this has improved thanks to Noel's work wrt the dynamic column creation
> magic <
> <a href="https://cgit.freedesktop.org/libreoffice/core/commit/">https://cgit.freedesktop.org/libreoffice/core/commit/</a>
> ?id=7282014e362a1529a36c88eb308df8ed359c2cfa ></span >

Still there is not a hard limit, only the lack of vertical column separators
reminds one that "something is missing" (i.e. columns, not the support to save
them). 
I think this is still not a strong enough protection, so we might want to
reopen.

<span class="quote">> I think we can close this as RESOLVED FIXED.
> @Gabor, please create a separate bug for the warning mentioned in <a href="show_bug.cgi?id=123100#c5">comment 5</a></span >

Would that be really necessary though?

I mean, imagine this workflow: user opens an existing XLS and we currently
allow them to add more than 256 columns of data, which will be lost simply
pressing Ctrl-S.

- If we stick to this option, we can warn them that saving in the current
format will cause data loss and recommend to choose another format, as Aron
proposed.
Then in the file save dialog the user still might or might not do so, and lose
their work (or not).

- If we were to introduce a hard limit on the number of columns (as Excel does)
there will be no way to add more columns than supported by XLS format and no
warning would be necessary.
This would not leave a space to err: if you need more columns, first convert
your spreadsheet.

I guess it's time to ask for UX advice on this.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>