I noticed a misunderstanding in the documentation:
In the view fields section, we should add the "Draggableviews: Content" field after the "Content: Title" field.
Hope it helps ;)

Comments

yepa created an issue. See original summary.

Happy_coding’s picture

Status: Active » Patch (to be ported)
StatusFileSize
new535 bytes

The fix

yepa’s picture

Assigned: yepa » Unassigned
damienmckenna’s picture

Priority: Normal » Minor
Status: Patch (to be ported) » Needs review

Thanks for the patch, hopefully someone will review it shortly.

A quick thing: please set the "status" field to "needs review" when you upload a patch, "patch (to be ported)" is for when there's a patch committed to one branch and it needs to be ported to another branch.

Thanks again!

jmuzz’s picture

Status: Needs review » Reviewed & tested by the community

I agree the documentation around this can use some work. This is a nice improvement.

istryker’s picture

So for the README example, this is good. However, I would like to get your thoughts. "Content" is the type of the view. If you have Taxonomy Term type view, then this changes to "Draggableviews: Taxonomy Term".

alison’s picture

Hi! Sorry, I'm still a little mixed up -- do we need the "weight" field for the drag tool to show up? That's what's happening to me right now -- I can totally open a new issue for that situation, but if it's actually not a bug and it's part of how to do it, I wanted to check with y'all first :)

istryker’s picture

@alisonjo2786 open a new issue. I'll commit this for now.

istryker’s picture

Status: Reviewed & tested by the community » Fixed

Committed

alison’s picture

Oof, blast from the past -- thanks for following up! I'll revisit the situation to see if I'm still having an issue, and submit a new one if necessary. Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.