Date: prev next · Thread: first prev next last
September 2017 Archives by date, by thread · List index


https://bugs.documentfoundation.org/show_bug.cgi?id=111843

V Stuart Foote <vstuart.foote@utsa.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|needsUXEval                 |
             Status|UNCONFIRMED                 |NEW
                 CC|                            |akshaydeepiitr@gmail.com,
                   |                            |erack@redhat.com
            Summary|active cell is not          |one active cell selection
                   |considered a single-cell    |is not searched with
                   |selection in find (&        |"Current selection only"
                   |replace) tool               |option set in Find &
                   |(clarification in comment   |Replace dialog tool
                   |1)                          |
     Ever confirmed|0                           |1

--- Comment #3 from V Stuart Foote <vstuart.foote@utsa.edu> ---
On Windows 10 Pro 64-bit en-US with
Version: 5.4.1.2 (x64)
Build ID: ea7cb86e6eeb2bf3a5af73a8f7777ac570321527
CPU threads: 8; OS: Windows 6.19; UI render: default; 
Locale: en-US (en_US); Calc: group

Confirming STR of the non-selection with just the currently active cell with
cursor focus, or the misbehavior of the "Current selection only" option which
shows active with just the one cell with cursor focus as a "range".

Selecting one additional cell produces valid search result. 

So seems like we have a logic flaw and test the view shell for HasSelection()
at [1] enabling the "Current selection only" option-with just the one cell with
cursor focus.  

Is it a valid selection range or not? 

   If it is--the search needs to accommodate a one cell range.

   If not--need to keep the Selection Btn disabled.

From UX perspective, searching just a single cell is not a common use, but why
not? Could be of some use in macro to conditionally test a cell.

=-ref-=
[1]
https://opengrok.libreoffice.org/xref/core/svx/source/dialog/srchdlg.cxx?a=true#915

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.