[Plone-UI] Review on content finder widget

[Dylan Jay]

On 18/01/2013, at 8:10 PM, Roché Compaan <roche at upfrontsystems.co.za> wrote:

> Hi Nathan
> 
> Thanks, this is great feedback.
> 
> I think we need a repo for the JS only part that's reusable outside
> plone, maybe simply call the repo contentfinder.js?

I think thats a good idea to keep the js as a standalone package, however how do we make it a dependency of p.a.widgets without it being an egg? We'd have to use git sub-packages or something, which isn't something plone normally uses.
Normally Plone packages solve this by using stable js packages and just copying the js in as a resource.

Maybe a more pragmatic approach is to start with the js in p.a.widgets while it's still immature. That way it can easily be improved without worrying about dependency tracking. Once the js reaches a point of maturity, it can be copied out to it's own package and then p.a.widgets could track it's progress from there.
I think p.a.widgets is a good first customer of the js code since we can start by using for referencefields but then use it for keyword fields too and then eventually use it as a general content browser. 

> 
> The plone integration can live in p.a.widgets.
> 
> --
> Roché Compaan
> Upfront Systems                   http://www.upfrontsystems.co.za
> 
> 
> On Fri, Jan 18, 2013 at 8:57 AM, Nathan Van Gheem <vangheem at gmail.com> wrote:
>> Hi everyone,
>> 
>> As we discussed, I would reach out to Alex Limi for thoughts on the latest:
>> http://static.nathanvangheem.com/plone-conf-2012-ui-sprint/clickdummies/contentfinder/index.html
>> 
>> Here are his responses, slightly edited:
>> 
>> - searching doesn't work. Just a click dummy thing but he suggested putting
>> dummy data in.
>> - suggests double clicking still allow you to navigate the folder--provide a
>> quicker way to navigate instead of clicking the arrow
>> - backspace when next to an entry in the text field should delete the
>> previous item. If this is too error-prone, maybe focus the item and make the
>> second backspace actually delete the item.
>> I like this.
>> - you are able to edit the "Click to search or browse" text
>> - It feels weird that I can't unselect selected items from the pulldown by
>> clicking on them again.
>> - Clicking outside the widget should dismiss the pulldown.
>> 
>> Also, we discussed:
>> 
>> - maybe a different arrow that highlights on hover
>> - nicer tooltip to show path
>> 
>> Roche, should we start a project to place these issues in? Put it in the
>> plone.app.widgets issue tracker perhaps?
>> 
>> 
>> -Nathan
> _______________________________________________
> UI mailing list
> UI at lists.plone.org
> https://lists.plone.org/mailman/listinfo/plone-ui