<div dir="ltr">Hi!<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 28, 2014 at 11:40 PM, Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org" target="_blank">manuq@laptop.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">2014-01-24 Sam Parkinson <<a href="mailto:sam.parkinson3@gmail.com">sam.parkinson3@gmail.com</a>><br>


<div class="im">><br>
> I worked on it for a little longer trying to make the star icons checkable, but there are lots of issues in doing so. In the end I think that it is very hard to implement and pretty worthless. Maybe a summary as a tooltip in the traditional design would be good, I'll try. Anyway, I really think the ticket is sort of fixed now; we now have check boxes in the journal. Also if you would like to see a bigger fix you can check out the sentence/expander view and the thumbnail view for the journal.<br>


><br>
> The (very messy and not PEP8) code is here: <a href="https://github.com/SAMdroid-apps/sugar/blob/fb8ec62cf308236fa292dd2635d4071800b3f3e1/src/jarabe/desktop/activitieslist.py" target="_blank">https://github.com/SAMdroid-apps/sugar/blob/fb8ec62cf308236fa292dd2635d4071800b3f3e1/src/jarabe/desktop/activitieslist.py</a>,<br>


<br>
</div>I tried to test your work replacing that file but got errors. Do you<br>
have a patch or diff against master?<br></blockquote><div><br></div><div>Oh, that might be a palette error. The palette invoker tries to get the path at the pointer and the icon_view replys differently to the tree_view :(  Try using this too: <a href="https://github.com/SAMdroid-apps/sugar-toolkit-gtk3/tree/journal-icon-view">https://github.com/SAMdroid-apps/sugar-toolkit-gtk3/tree/journal-icon-view</a></div>

<div><br></div><div>The palette issue is:</div><div> 1. Making stuff not crash: the tree_view is different api wise to the icon_view</div><div> 2. Making the palette actually show: this was my issue</div><div><br></div><div>

And the whole thing with the stars is weird. The events like toggle don't seem to be passed on normally to the cell renderer. The (gtk) model code also has a major bug that prevents you from changing the values of a inserted row (a name error or something funny).</div>

<div><br></div><div>Thanks,</div><div>Sam</div></div></div></div>