This looks really good Jake, great work!

I do have one small suggestion; In the rave.js it looks like you can
abstract out some common functionality to a helper function.

Aaron

On Wed, Jan 25, 2012 at 9:30 AM, Jacob Hilker (Commented) (JIRA) <
[email protected]> wrote:

>
>    [
> https://issues.apache.org/jira/browse/RAVE-263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13193064#comment-13193064]
>
> Jacob Hilker commented on RAVE-263:
> -----------------------------------
>
> I have submitted a patch for this on the Review Board
>
> https://reviews.apache.org/r/3622/
>
> > drag & drop target column heights
> > ---------------------------------
> >
> >                 Key: RAVE-263
> >                 URL: https://issues.apache.org/jira/browse/RAVE-263
> >             Project: Rave
> >          Issue Type: Bug
> >          Components: rave-portal
> >    Affects Versions: 0.3-INCUBATING
> >            Reporter: Matt Franklin
> >
> > The target column heights during the drag & drop operation are too small.
>
> --
> This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA
> administrators:
> https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
> For more information on JIRA, see: http://www.atlassian.com/software/jira
>
>
>

Reply via email to