<div dir="ltr"><div>You forgot this one: <a href="https://bugs.kde.org/show_bug.cgi?id=377857">https://bugs.kde.org/show_bug.cgi?id=377857</a><br></div>It is related too<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Best regards, <br></div></div></div></div>
<br><div class="gmail_quote">On Tue, Jul 25, 2017 at 10:32 PM, Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Relevant entries in bugzilla :<br>
<br>
LT :<br>
<br>
<a href="https://bugs.kde.org/show_bug.cgi?id=148572" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.<wbr>cgi?id=148572</a><br>
<br>
PreviewView:<br>
<br>
<a href="https://bugs.kde.org/show_bug.cgi?id=336593" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.<wbr>cgi?id=336593</a><br>
<a href="https://bugs.kde.org/show_bug.cgi?id=380331" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.<wbr>cgi?id=380331</a><br>
<a href="https://bugs.kde.org/show_bug.cgi?id=302559" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.<wbr>cgi?id=302559</a><br>
<br>
Gilles Caulier<br>
<br>
2017-07-26 6:23 GMT+02:00 Gilles Caulier <<a href="mailto:caulier.gilles@gmail.com">caulier.gilles@gmail.com</a>>:<br>
> Hi Andrey,<br>
><br>
> This have been a long history in bugzilla about to be able to display<br>
> or edit some properties over the canvas (editor and preview mode), as<br>
> rating, tags, or labels.<br>
><br>
> The result is a shared point to implement or not this feature, as the<br>
> possibility already exist every where : context menu, sidebar, and<br>
> keyboard shortcuts.  And of course nothing have be done as it's<br>
> delcate to take a clear decision about.<br>
><br>
> But, something as been implemented in slideshow, as only keyboard<br>
> shortcuts was available to apply main properties to images. On the<br>
> botoom corner of the image while slide show, if the right option is<br>
> turned on in setup dialog, you are able to rate, tags, or change color<br>
> labels.So, the techical solution exists in slide show widget, and can<br>
> be backported to image preview.<br>
><br>
> But take a care, it's not the most simple task to do to start coding<br>
> in digiKam and to learn the implementation.<br>
><br>
> If rate/tags/labels will become available over canvas, well all of<br>
> then must be optional in setup preview section, to be homogeneous.<br>
> this will touch more code.<br>
><br>
> Also, the preview canvas, is also used in Light table and must be also<br>
> patched to provide this feature.<br>
><br>
> Even that you are right. The preview canvas, is partially implemented<br>
> in imagepreviewview.cpp. This is the derivated class to manage the<br>
> preview with F3 key from album view. There are parent classes in<br>
> core/libs/widgets/<wbr>graphicsview/. The preview view is an item loaded in<br>
> canvas and implemented with imagepreviewviewitem.cpp (it's Qt<br>
> model/view based).<br>
><br>
> Something similar is implemented with Light Table with some difference<br>
> and customizations. Mostly the main code is located in core lib<br>
> classes, and customization is located in derivated classes. The parent<br>
> classes are free of database to be usable in Showfoto as this one<br>
> don't use the database. So with your proposal, the code to change is<br>
> mandatory located in customization classes.<br>
><br>
> Now the implementation from slideshow to be able to show and edit<br>
> rate/tags/labels over the preview is located in<br>
> core/utilities/slideshow/<wbr>slideosd.cpp.There are 3 magic widgets which<br>
> perform 90 of the job with the properties to show/edit : RatingWidget,<br>
> ColorLabelSelector, and PickLabelsSelector. This one are usable as<br>
> well without change as it's done with slideosd.cpp.<br>
><br>
> But the real question, is more ambitious : why not to reuse as well<br>
> the slideosd implementation as an overlay widget for preview canvas.<br>
> This will be homogenous everwhere and factorize/simplify all codes<br>
><br>
> So the questions are open...<br>
><br>
> Best<br>
><br>
> Gilles Caulier<br>
><br>
> 2017-07-26 1:03 GMT+02:00 Andrey Goreev <<a href="mailto:aegoreev@gmail.com">aegoreev@gmail.com</a>>:<br>
>> Hello,<br>
>><br>
>> I want to make an image star rating visible in image preview view and<br>
>> wondering which file should I be modifying.<br>
>> I am looking at imagepreviewview.cpp in<br>
>> <a href="https://github.com/KDE/digikam/tree/master/app/views" rel="noreferrer" target="_blank">https://github.com/KDE/<wbr>digikam/tree/master/app/views</a><br>
>> Is this the right one? Any tip would be greatly appreciated.<br>
>><br>
>> Best regards,<br>
>> Andrey<br>
</blockquote></div><br></div>