Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sortable images on media multiple #28

Merged
merged 9 commits into from
Jan 5, 2016
Merged

Sortable images on media multiple #28

merged 9 commits into from
Jan 5, 2016

Conversation

motchju
Copy link
Contributor

@motchju motchju commented Dec 31, 2015

Sortable functionality on Media Module for multiple image.

Adding sortable migration
Route added + logic
Order by is now added
Image is now wrapped inside the <figure> tags
Load query ui as it’s needed for the sortable functionality.
Data-id added, needed if we reorder without refreshing the page.
Js logic and ajax call to store the order
Data id added on figure tags
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;

class AddSortable extends Migration

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Each class must be in a namespace of at least one level (a top-level vendor name)

Js change for the helper class + some css added
Clean up of inline css
Jquery sortable, axis :’x’ removed, needed if the images used at least
2 lines
@nWidart
Copy link
Member

nWidart commented Jan 5, 2016

Thank for this awesome addition! 😄

nWidart added a commit that referenced this pull request Jan 5, 2016
Sortable images on media multiple
@nWidart nWidart merged commit 8619c9f into AsgardCms:master Jan 5, 2016
zedee pushed a commit to zedee/Media that referenced this pull request Mar 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants