Bug 320708 - Remove filter when action is executed
Summary: Remove filter when action is executed
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: contactlist (show other bugs)
Version: git-latest
Platform: Other Linux
: NOR wishlist
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-04 11:47 UTC by Alex Fiestas
Modified: 2013-10-25 11:48 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Fiestas 2013-06-04 11:47:54 UTC
the feature is simple, remove the filter once you open a chat (or execute any action)
the rationale behind it is simple as well, when you filter you do that because you have a concrete reason, like finding david edmunson
once I find you, and I execute an action with you (open chat, open kbattleship, etc) the filter should be removed

Reproducible: Always
Comment 1 David Edmundson 2013-06-05 18:39:12 UTC
+1 from me.
I'll let Martin confirm the bug if he's happy with it too.
Comment 2 Martin Klapetek 2013-06-13 12:49:59 UTC
Should it just remove the filter string or hide the filter bar too?
Comment 3 Alex Fiestas 2013-06-13 14:23:37 UTC
Imho hide it too,
Comment 4 Daniele E. Domenichelli 2013-06-13 15:08:33 UTC
I'm not a big fan of this Idea, if you just need to start one action you can just use the krunner...
When I'm testing something, I often keep the filter on one account of mine and just keep starting actions on that. 
If you implement this, I suggest adding some "stick" button to the filter bar, so that you can opt-out this feature when you don't want it.
Comment 5 Alex Fiestas 2013-06-13 15:20:34 UTC
Well, the interface should be optimized for those users that do not even know that KRunner exists, after all it is still a hidden feature (you need to know the shortcut).

About the stick button, if the filterbar was big it wouldn't add too much noise but being usually quite narrow.. I'm not sure.

This is a typical case of priorities, by hiding it by default you will probably make life easier to newbie users of the contactlist (like myself) but making it harder to hardcore users.

(In reply to comment #4)
> I'm not a big fan of this Idea, if you just need to start one action you can
> just use the krunner...
> When I'm testing something, I often keep the filter on one account of mine
> and just keep starting actions on that. 
> If you implement this, I suggest adding some "stick" button to the filter
> bar, so that you can opt-out this feature when you don't want it.
Comment 6 Alex Fiestas 2013-06-13 15:21:23 UTC
I forgot, can you imagine an use case that can be applied to users, even if they are advanced? the use case you have given is quite specific to developers
Comment 7 Daniele E. Domenichelli 2013-06-13 16:09:14 UTC
(In reply to comment #5)
> About the stick button, if the filterbar was big it wouldn't add too much
> noise but being usually quite narrow.. I'm not sure.

If you have this feature, then you probably don't need the close button anymore, perhaps we can just replace it with a stick button when the field is not empty...


(In reply to comment #6)
> I forgot, can you imagine an use case that can be applied to users, even if
> they are advanced? the use case you have given is quite specific to
> developers

You choose a contact, then you start some action (let's say you send a file), and then you want to start another one with the same contact (let's say you want to send another file)
Comment 8 Martin Klapetek 2013-06-13 16:34:30 UTC
I thought about repeated actions. I believe that in most cases you are also chatting with that person and as such you can repeatedly start the same action from the text-ui, which is even easier as it's in a toolbar. Would that satisfy that?
Comment 9 David Edmundson 2013-06-13 16:49:19 UTC
I'm not sure.

Given the patch is ready, let's try it first before we come to any conclusions about whether we like it and what we need to change.
Comment 10 Martin Klapetek 2013-06-13 18:49:44 UTC
Alright, I'll commit it and leave this bug open for now. I'll put this in stable too, so we can get some feedback earlier.
Comment 11 David Edmundson 2013-06-13 18:50:25 UTC
as in 0.6.x?
Comment 12 Martin Klapetek 2013-06-13 18:51:14 UTC
Well, do/will we have enough stuff to release 0.6.3?
Comment 13 David Edmundson 2013-06-13 18:56:53 UTC
Possibly. There are some patches, nothing worth it at the moment.

In either case I don't want this in 0.6.
1) We don't know if it's great or whether it'll suck
2) it's a new feature. Rules are against that. Behaviour shouldn't change in minor releases.
Comment 14 Martin Klapetek 2013-06-13 19:20:16 UTC
Git commit e0e5d290c5468ca66533907689de61310447ed86 by Martin Klapetek.
Committed on 13/06/2013 at 21:19.
Pushed by mklapetek into branch 'master'.

Automatically hide the filter bar after an action has been started

As this makes repeatable actions harder to execute, this patch is mainly
for testing purposes, it may be reverted before 0.7, see the bug for
details
FIXED-IN: 0.7.0
REVIEW: 110998

M  +12   -0    contact-list-widget.cpp
M  +1    -1    contact-list-widget.h
M  +6    -0    filter-bar.cpp
M  +3    -0    filter-bar.h
M  +3    -0    main-widget.cpp

http://commits.kde.org/telepathy-contact-list/e0e5d290c5468ca66533907689de61310447ed86