sugarlabs / chat

The Sugar chat activity
GNU General Public License v2.0
5 stars 24 forks source link

Feature: searching #18

Closed quozl closed 5 years ago

quozl commented 5 years ago

One of the original design features not yet implemented is searching.

Reference: http://wiki.laptop.org/go/Chat

nswarup14 commented 5 years ago

@quozl Wouldn't it better if the chat history was preserved in the journal entry itself? As I understand, the proposed search feature is to highlight the relevant text that matches with the search query in the chat activity itself.

Assuming a chat room with 4 participants, if one of them happens to leave, the chat log is stored in the journal. If the participant later opens the journal object, the earlier chat log is loaded, as expected. Thus we are able to achieve persistence storage, even if the participants continue to leave and rejoin as many number of times.

The only time the chat log is lost is when the journal object is deleted. This is once again expected, but we needn't store the chat log of an activity instance whose journal object is deleted.

Unless I have misunderstood the proposed feature itself, I feel that preserving chat in the journal entry is sufficient. Please feel free to correct me.

Thanks

quozl commented 5 years ago

Thanks for thinking about this. I neglected to fully express my thinking or think through the reasons. I've edited the initial comment.

quozl commented 5 years ago

1ecad5a merged without complete position support.