[Kde-pim] Review Request 117849: Improve KMail's Create Todo/Event widgets

[Kevin Ottens]

> On April 28, 2014, 5 p.m., Kevin Ottens wrote:
> > messageviewer/widgets/todoedit.cpp, line 218
> > <https://git.reviewboard.kde.org/r/117849/diff/1/?file=269052#file269052line218>
> >
> >     Please don't! That's breaking my workflow.
> >     
> >     There's two uses of this widget. A quick "reply later" where you basically just hit enter then close via escape. Or you're scanning the email for action items in there, so you type something hit enter, type something hit enter, etc. when done you hit escape.
> 
> Dan Vrátil wrote:
>     >> you're scanning the email for action items in there, so you type something hit enter, type something hit enter, etc. when done you hit escape.
>     This is just wrong:
>     
>     1) you hit enter and *nothing* happens. Absolutely nothing. First time I created 5 identical todos, because I kept hitting enter hoping something will happen eventually. You need show users that something happened. Closing the widgets is analogous to saying "Done".
>     
>     2) EventEdit hides when you type something and hit enter. Inconsistency. Either both hide, or none of them.
>     
>     
>     Finally, adapting your workflow to "type something, hit enter, press Ctrl+T, type something, hit enter, press Ctrl+T" does not sound that painful.
>     
>

1) Well, then user feedback is needed indeed. Doesn't necessarily have to make the widget bigger though if that's something temporary (KMessageWidget, notification, passive popup are possibilities I guess).

2) Different use cases so I'm not sure it has to be mandatory to have them behave exactly the same way, but I see the argument. If we want to go that route, I'd uniformize toward the behavior of TodoEdit though. So that you can scan the email for events and enter them one after another. Again, I'm not sure that's something we want (even the interaction schemes of both widgets are very different, you're more likely to fiddle around in several fields for the events than for the todos).


- Kevin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117849/#review56775
-----------------------------------------------------------


On April 28, 2014, 3:45 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117849/
> -----------------------------------------------------------
> 
> (Updated April 28, 2014, 3:45 p.m.)
> 
> 
> Review request for KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> Couple improvements to the widgets:
> 
> 1) added "Save" button (you can still create event/todo just by hitting enter in the edit line, but that's not easily discoverable (took me a while to figure that out)
> 2) removed the close tool button and instead added "Close" button next to the "Save" one, so that we create line of buttons
> 3) added "Open Editor" button which opens the full-features incidence-editor. This is useful when you want to sent for instance a reminder - otherwise you would have to start KOrganizer to edit the just created event
> 4) fixed TodoEdit to actually close after accepting
> 5) added label to the collection combobox
> 6) slightly modified the layout
> 7) added code to EventEdit to automatically update endDateTimeEdit, when a change in startDateTimeEdit would cause the dtEnd to be before dtStart (unfortunately that does not work because the KDateTimeEdit is broken, I'll put up a patch for kdelibs later).
> 
> 
> Diffs
> -----
> 
>   messageviewer/CMakeLists.txt aa9034c 
>   messageviewer/tests/eventedittest.cpp 0c7ea35 
>   messageviewer/tests/todoedittest.cpp e8fe51e 
>   messageviewer/widgets/eventedit.h c1b6a99 
>   messageviewer/widgets/eventedit.cpp be52916 
>   messageviewer/widgets/todoedit.h f0d0774 
>   messageviewer/widgets/todoedit.cpp 472ba00 
> 
> Diff: https://git.reviewboard.kde.org/r/117849/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> Before
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/04/28/8f9b5c1a-b71c-4805-b4ad-d0f3f644df18__kmail-old.png
> Now
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/04/28/a0647751-b97b-4a75-a652-85bf6ba6667d__kmail-new.png
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/