Closed (fixed)
Project:
DraggableViews
Version:
8.x-1.x-dev
Component:
Documentation
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
6 Jun 2018 at 16:49 UTC
Updated:
15 Feb 2019 at 15:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
Happy_coding commentedThe fix
Comment #3
yepaComment #4
damienmckennaThanks 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!
Comment #5
jmuzz commentedI agree the documentation around this can use some work. This is a nice improvement.
Comment #6
istryker commentedSo 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".
Comment #7
alisonHi! 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 :)
Comment #8
istryker commented@alisonjo2786 open a new issue. I'll commit this for now.
Comment #10
istryker commentedCommitted
Comment #11
alisonOof, 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!