Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/#review88863 --- applets/taskmanager/plugin/backend.cpp (line 364)

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/#review88864 --- Perhaps there should be a title action, similar to the Recent

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Sebastian Kügler
> On Nov. 26, 2015, 2:08 p.m., Eike Hein wrote: > > Perhaps there should be a title action, similar to the Recent Documents one? For which purpose? (I think the distinction between jump list and 'task option' is purely technical, no user benefit in cluttering up a context menu with a title

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Eike Hein
> On Nov. 26, 2015, 2:08 p.m., Eike Hein wrote: > > Perhaps there should be a title action, similar to the Recent Documents one? > > Sebastian Kügler wrote: > For which purpose? (I think the distinction between jump list and 'task > option' is purely technical, no user benefit in

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Hugo Pereira Da Costa
> On Nov. 26, 2015, 2:08 p.m., Eike Hein wrote: > > Perhaps there should be a title action, similar to the Recent Documents one? > > Sebastian Kügler wrote: > For which purpose? (I think the distinction between jump list and 'task > option' is purely technical, no user benefit in

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/ --- (Updated Nov. 26, 2015, 6:48 nachm.) Review request for Plasma, KDE

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Thomas Pfeiffer
> On Nov. 26, 2015, 2:08 p.m., Eike Hein wrote: > > Perhaps there should be a title action, similar to the Recent Documents one? > > Sebastian Kügler wrote: > For which purpose? (I think the distinction between jump list and 'task > option' is purely technical, no user benefit in

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/#review88874 --- Ship it! Ship It! - Eike Hein On Nov. 26, 2015, 6:48

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-26 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/ --- (Updated Nov. 26, 2015, 8:03 p.m.) Status -- This change has been

Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-25 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/ --- Review request for Plasma, KDE Usability and Eike Hein. Bugs: 339750

Re: Review Request 126174: [Task Manager] Add support for Jump Lists

2015-11-25 Thread Thomas Pfeiffer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126174/#review88858 --- I don't see any usability problems here. If we add this to