Review Request 119250: Events and trips according to date ranges

Sinny Kumari ksinny at gmail.com
Sun Jul 20 15:26:27 UTC 2014


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


Works fine now. Just two minor fix:
1. New events with blank name shouldn't be created. Save button should be disabled until name some valid name is enterd
2. Creating a new event with an existing event name ends up editing same. Ideally, it should give error message saying event already exits.

- Sinny Kumari


On July 20, 2014, 3:08 p.m., Shantanu Tushar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119250/
> -----------------------------------------------------------
> 
> (Updated July 20, 2014, 3:08 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> I got this idea on a recent trip that usually a family would have a device where they store their trips/events photos on. Easier than tagging everything, what if you could just say "Hey, I went to Kashmir during 5 April to 14 April 2014" and voila!
> 
> This backend implements this idea in a (hopefully) easy-to-use way. I'll love to hear some feedback around this :)
> 
> 
> Diffs
> -----
> 
>   browsingbackends/metadatabackends/eventsbackend/eventsfiltermodel.cpp PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventsmodel.h PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventsmodel.cpp PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventspicturesmodel.h PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventspicturesmodel.cpp PRE-CREATION 
>   browsingbackends/onlineservices/picasa/picasabackend.h 83b3eb2 
>   browsingbackends/onlineservices/picasa/picasabackend.cpp b01ba4a 
>   browsingbackends/onlineservices/picasa/picasacomponents/PicasaSidePanel.qml d15a46e 
>   libs/mediacenter/abstractbrowsingbackend.h 14ad502 
>   libs/mediacenter/abstractbrowsingbackend.cpp 6cd32f8 
>   libs/mediacenter/backendsmodel.cpp 2823fb6 
>   libs/mediacenter/filtermediamodel.h b8272f6 
>   mediaelements/mediabrowser/MediaBrowser.qml b4ebdf9 
>   shells/newshell/package/contents/ui/mediacenter.qml 15c9cee 
>   browsingbackends/metadatabackends/CMakeLists.txt ae56603 
>   browsingbackends/metadatabackends/eventsbackend/CMakeLists.txt PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventsbackend.cpp PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventsbackend.desktop PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventsbackend.h PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventscomponents/DatePicker.qml PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventscomponents/Digit.qml PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventscomponents/EventsConfiguration.qml PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventscomponents/qmldir PRE-CREATION 
>   browsingbackends/metadatabackends/eventsbackend/eventsfiltermodel.h PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/119250/diff/
> 
> 
> Testing
> -------
> 
> Seems to work as expected, I will be attaching screenshots.
> 
> 
> File Attachments
> ----------------
> 
> Showing events
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/07/12/c4c99bbd-f77f-4167-922e-79f90949040b__events.png
> Showing photos from an event
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/07/12/a4ed7663-64b6-4963-aed3-5e41cc8e51ee__showing-event.png
> Editing dates of an existing event
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/07/12/965b667b-e209-44a3-bc64-95f81b18913e__editing-event.png
> 
> 
> Thanks,
> 
> Shantanu Tushar
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140720/5ebaa6b6/attachment.html>


More information about the Plasma-devel mailing list