Issue 111984 - [CWS findbar01] Testtool: Hidden toolbar controls appear as visible
Summary: [CWS findbar01] Testtool: Hidden toolbar controls appear as visible
Alias: None
Product: General
Classification: Code
Component: ui (show other issues)
Version: DEV300m78
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
Depends on:
Reported: 2010-06-01 09:26 UTC by joerg.skottke
Modified: 2017-05-20 10:55 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Note You need to log in before you can comment on or make changes to this issue.
Description joerg.skottke 2010-06-01 09:26:55 UTC
The button "Search Dialog" is hidden on the FindBar by default and must be
enabled via context menu "visible icons->Search Dialog" (or similar).

When accessing the toolbar in its default configuration (Search Dialog button
hidden) using the mehods .getItemCount(), .isVisible() and .exists() we get
true/the number of items including the hidden one in any case.

.isVisible() should return false
.getItemCount() should return the number of visible items only
Comment 1 joerg.skottke 2010-06-01 09:27:32 UTC
cc CD
Comment 2 joerg.skottke 2010-06-02 11:02:37 UTC
Removed this task from CWS findbar01, submitted issue 112026 as follow-up and
instruction on how to proceed.
Comment 3 gregor.hartmann 2010-06-04 14:19:42 UTC

<Button>.isvisible  will always fail and 
<Button>.exists will always be false (this is a bug, should also fail currently)

print TextObjectBar.isvisible (Hochgestellt)
print TextObjectBar.exists( Hochgestellt )

will both return true, as only the toolbar is asked and the additional parameter
is ignored (should fail also)

If I change get ItemCount I will also have to change the numbering of the
buttons which would mean that you have to change scripting where the buttons are
called by position.

I can do it, did it for Menus also.

Comment 4 joerg.skottke 2010-06-06 19:16:29 UTC
Sounds good. I'm not using icons by index, and i don't think others do, it is a
randomizer anyway. But we will need to check that to be sure. 
Adding TBO to cc.
Comment 5 thorsten.ziehm 2010-11-09 10:17:55 UTC
OOo 3.3 is nearly final. I change the target of this issue to OOo 3.x. Please
find a solution and set a correct target, if you know when a fix can be integrated.
Comment 6 Marcus 2017-05-20 10:55:36 UTC
Reset assigne to the default "".