Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#199 closed bug (fixed)

[Terminal] Find feature is missing

Reported by: diver Owned by: Niels.Reedijk@…
Priority: normal Milestone: R1
Component: Applications Version:
Keywords: Cc: Niels.Reedijk@…
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

Find feature in Terminal is missing

Attachments (5)

slider.PNG (9.7 KB) - added by jackburton 13 years ago.
Missing slider invalidation
199-1.patch (8.6 KB) - added by Niels.Reedijk@… 13 years ago.
GUI portion of the patch
199-2.patch (13.1 KB) - added by Niels.Reedijk@… 13 years ago.
Updated GUI patch
199-3.patch (19.6 KB) - added by Niels.Reedijk@… 13 years ago.
Final (?) patch
199-4.patch (20.4 KB) - added by Niels.Reedijk@… 13 years ago.
Second final attempt

Download all attachments as: .zip

Change History (24)

comment:1 Changed 13 years ago by axeld

Owner: changed from sikosis to axeld

comment:2 Changed 13 years ago by korli

bug_group: developers

comment:3 Changed 13 years ago by diver

Summary: Find feature in Terminal is missing[Terminal] Find feature is missing

comment:4 Changed 13 years ago by diver

(In reply to comment #1)

Created an attachment (id=265) [edit] Missing slider invalidation

I guess you meant this attach for #193, right? ;-)

comment:5 Changed 13 years ago by jackburton

ARGH Sorry :)

comment:6 Changed 13 years ago by jackburton

attachments.isobsolete: 01

Changed 13 years ago by jackburton

Attachment: slider.PNG added

Missing slider invalidation

comment:7 Changed 13 years ago by Niels.Reedijk@…

Cc: stefano.ceccherini@… added

comment:8 Changed 13 years ago by Niels.Reedijk@…

I'm going to try this one. Stefano, CC'ing you for review.

comment:9 Changed 13 years ago by jackburton

Looks good, except for some style issues I already told you about on irc. At least it doesn't do any harm :)

Changed 13 years ago by Niels.Reedijk@…

Attachment: 199-1.patch added

GUI portion of the patch

comment:10 Changed 13 years ago by Niels.Reedijk@…

attachments.isobsolete: 01

comment:11 Changed 13 years ago by Niels.Reedijk@…

Cc: Niels.Reedijk@… added

Changed 13 years ago by Niels.Reedijk@…

Attachment: 199-2.patch added

Updated GUI patch

comment:12 Changed 13 years ago by jackburton

Owner: changed from axeld to Niels.Reedijk@…

comment:13 Changed 13 years ago by Niels.Reedijk@…

Status: newassigned

comment:14 Changed 13 years ago by Niels.Reedijk@…

attachments.isobsolete: 01

comment:15 Changed 13 years ago by Niels.Reedijk@…

Saw one thing I'll correct: the UseHeaders [ FDirName $(HAIKU_TOP) src kits tracker ] ; line in the Jamfile is unnecessary and was for testing purposes. I'll revert that one.

Other note: R5's terminal includes a regexp search. I did not implement that, due to the fact that it would make the code a lot more complicated. Perhaps if BString is extended to do regexp searches. If it is important to add it, it deserves its own bug.

comment:16 Changed 13 years ago by Niels.Reedijk@…

attachments.isobsolete: 01

Changed 13 years ago by Niels.Reedijk@…

Attachment: 199-3.patch added

Final (?) patch

Changed 13 years ago by Niels.Reedijk@…

Attachment: 199-4.patch added

Second final attempt

comment:17 Changed 13 years ago by Niels.Reedijk@…

Sending terminal/Changelog Deleting terminal/Constants.h Sending terminal/FindDlg.cpp Sending terminal/FindDlg.h Sending terminal/Jamfile Sending terminal/TermBuffer.cpp Sending terminal/TermBuffer.h Sending terminal/TermConst.h Sending terminal/TermView.cpp Sending terminal/TermView.h Sending terminal/TermWindow.cpp Sending terminal/TermWindow.h Transmitting file data ........... Committed revision 18561.

comment:18 Changed 13 years ago by Niels.Reedijk@…

Resolution: fixed

comment:19 Changed 13 years ago by Niels.Reedijk@…

Status: assignedclosed
Note: See TracTickets for help on using tickets.