[Kde-pim] Review Request: (Korganizer) Give the user a message saying he must configure work days to use the work view

[Allen Winter]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/884/#review1386
-----------------------------------------------------------

Ship it!


I like my message better.  Your idea to change "Work Hours" to "Work Period" is good.


trunk/KDE/kdepim/korganizer/koviewmanager.cpp
<http://reviewboard.kde.org/r/884/#comment875>

    How about:
    "Unable to display the work week view since there are no work days configured. Please properly configure at least 1 work day in the Time and Date preferences."


- Allen


On 2009-06-25 13:22:06, Sergio Martins wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/884/
> -----------------------------------------------------------
> 
> (Updated 2009-06-25 13:22:06)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> Here's the string for review:
> 
> "In order to use this view you must configure work days under Time and Date preferences."
> 
> 
> Another thing we could fix:
> In the time prefs dialog, I think we should change "Working Hours" to "Working Period" or something else because we can configure days there.
> 
> 
> This addresses bug 192239.
>     https://bugs.kde.org/show_bug.cgi?id=192239
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdepim/korganizer/koviewmanager.cpp 987197 
> 
> Diff: http://reviewboard.kde.org/r/884/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergio
> 
>

_______________________________________________
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/