Issue 46270 - "Find All" button in Find&Replace dialog is always disabled in Draw and Impress thus should be hidden
Summary: "Find All" button in Find&Replace dialog is always disabled in Draw and Impre...
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: editing (show other issues)
Version: OOo 2.0 Beta
Hardware: All All
: P4 Trivial (vote)
Target Milestone: OOo 3.3
Assignee: stefan.baltzer
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-03-30 08:14 UTC by dazzling0126
Modified: 2010-05-26 17:05 UTC (History)
6 users (show)

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


Attachments
"Find All" is always not activated. (317.77 KB, image/jpeg)
2005-03-30 08:15 UTC, dazzling0126
no flags Details
proposed fix (1.02 KB, patch)
2009-06-12 07:20 UTC, dtardon
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description dazzling0126 2005-03-30 08:14:23 UTC
Dear All.

When I click [Edit]-[Find & Replace],
"Find All" button is always not activated.
So I cannot search the word at the same time.

I attach capture file of bug, so check it, please.
Thanks :)
Comment 1 dazzling0126 2005-03-30 08:15:21 UTC
Created attachment 24399 [details]
"Find All" is always not activated.
Comment 2 wolframgarten 2005-03-30 08:43:58 UTC
Reproducible.
Comment 3 wolframgarten 2005-03-30 08:44:20 UTC
Reassigned.
Comment 4 groucho266 2005-04-18 09:38:34 UTC
"Find All" is for highlighting all occurences of the search string in the
document.  This is not possible in Draw or Impress because the pages of the
document are not visible at the same time in those views where the text can be
edited.  Therefore the button is disabled.

It would be technically possible to remove the button when this feature is not
supported. However, this would require a lot of work and might introduce a lot
of regression.

Therefore I think it is better to leave it as it is.  Maybe put a remark into
the help text.
Comment 5 groucho266 2005-04-18 09:39:34 UTC
Reopening.
Comment 6 groucho266 2005-04-18 09:41:51 UTC
Back to QA.
Comment 7 groucho266 2005-04-18 09:43:43 UTC
Re-setting resolution to wontfix.
Comment 8 wolframgarten 2005-04-19 08:00:20 UTC
Closed.
Comment 9 dtardon 2009-06-12 07:19:54 UTC
dtardon->af: Unless I have missed something important, correction of that
doesn't seem to be so hard a thing. It's just a matter of hiding the 'Find All'
button and moving the 'Find' button a little. There is already _a lot_ of button
hiding and position correcting in the implementation of the dialog, thus I guess
one more cannot hurt.

Could you reconsider fixing this issue?
Comment 10 dtardon 2009-06-12 07:20:44 UTC
Created attachment 62931 [details]
proposed fix
Comment 11 dtardon 2009-06-12 07:25:58 UTC
.
Comment 12 caolanm 2009-06-12 10:12:19 UTC
lets reopen to get an opinion on that
Comment 13 caolanm 2009-06-12 10:12:56 UTC
cmc->af: what do you think, given that dtardon has cooked up a patch for it
Comment 14 stefan.baltzer 2010-03-05 12:05:12 UTC
SBA: Adjusting summary. 
+1 from my side to hide that button for Draw and Impress as there is no resource
in sight to implement that functionality OR to re-invent that dialog in large
scale (i.e. have UI to ease search for line break and the like like Word has to
avoid regex study is an old request (don't know the issue ID).
The F&R dialog looks a little different in the apps anyway.

Note that some check boxes (Whole words, Similarity search, Current selection
only) behave "useless" in Draw and Impress, too. But an estimation of the effort
to make those work (before hiding them "in one go") sounds feasible.
Comment 15 caolanm 2010-04-08 10:50:00 UTC
I don't think there's any objections against doing this then, right ?. Given
that, how about I take care of this.
Comment 16 caolanm 2010-04-08 20:29:03 UTC
committed to sdfindall
Comment 17 caolanm 2010-04-13 17:35:06 UTC
reassign for qa
Comment 18 stefan.baltzer 2010-05-06 07:47:04 UTC
Adjusting summary to reflect what has been done (hide that button).
Verified in CWS sdfindall.
Comment 19 caolanm 2010-05-26 17:05:22 UTC
closing, integrated DEV300_m79