Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
Are there any plans on creating a D8 version of this module?
Hi,
Are there any plans on creating a D8 version of this module?
Comments
Comment #1
Lukas von BlarerDid anyone start working on this?
Comment #2
kingfisher64 CreditAttribution: kingfisher64 commentedI'd like to know this as well :)
Comment #3
minorOffense CreditAttribution: minorOffense at Coldfront Labs Inc. commentedThere will be. I spent a day a year ago trying out some D8 stuff but nothing materialized.
It'll be a straight port. The big missing piece for me at the time was how to convert the ctools exportable optionsets into the new CMI system. There wasn't an upgrade path / procedure. There might be now.
If anyone wants to give it a shot let me know, I can grant access to the module on GitHub.
Comment #4
kingfisher64 CreditAttribution: kingfisher64 commentedhttps://www.drupal.org/project/drupalmoduleupgrader is what is supposed to help with upgrading. :)
Comment #5
minorOffense CreditAttribution: minorOffense at Coldfront Labs Inc. commentedYeah, like I said, didn't exist a year ago ;-)
If you want to give it a run through try it out. Let me know how it goes. I don't plan on creating a D8 version until the fall at least. Or until the RC's are out.
Comment #6
manishmore CreditAttribution: manishmore as a volunteer and commentedI would be like to work for D8 version.
Comment #7
bloomt CreditAttribution: bloomt commentedHas anyone started porting this module to drupal 8?
Comment #8
ChristianP CreditAttribution: ChristianP commentedNo response in 2 months. Just checking in if anyone knows anything about the Drupal 8 version?
Comment #9
amaria CreditAttribution: amaria commentedTyring it now in my sandbox.
Comment #10
amaria CreditAttribution: amaria commentedSo I was pretty far into the conversion when I realized that the api needs to be rethought. Right now flexslider_add is using drupal_add_js and drupal_add_css to add the library and js settings. However these functions have been removed in favor of #attached. In that case, the idea of simply adding the library configuration arbitrarily is no longer viable as a simple function call.
Any thoughts on replacing the api?
In the meantime, I can proceed without it.
Comment #11
amaria CreditAttribution: amaria commentedSo I have a working D8 version of this in my sandbox. It's pretty much a straight port. There are still a few things to work out...
But it works so please have a look and let me know what you think!
BTW, I dealt with the API (flexslider_add) by changing it to return an #attached array that can be added to any render array.
Comment #12
amaria CreditAttribution: amaria commentedThe latest in my sandbox is pretty much fully functional. Still some things to do before it's complete and ready for review...
Hopefully someone will try it out :)
Comment #13
minorOffense CreditAttribution: minorOffense at Coldfront Labs Inc. commentedFor #10 If
#attached
works on blocks in D8 then yeah we should do that instead.Comment #14
minorOffense CreditAttribution: minorOffense at Coldfront Labs Inc. commentedI've copied your code into the 8.x-2.x branch of the module so people can start trying the code out. Anyone reading this comment be sure to note that the code is not ready for production use and API changes may still occur.
Comment #15
amaria CreditAttribution: amaria commentedOk, Great!
Comment #16
bloomt CreditAttribution: bloomt commentedYour sandbox release is working good for me, haven't run into any bugs yet
Comment #17
bloomt CreditAttribution: bloomt commentedFound a bug,
I cannot add a flexslider field inside a view. Actually I can add the field but then I cannot go back and edit it.
Comment #18
bloomt CreditAttribution: bloomt commentedAlso I get "The website encountered an unexpected error. Please try again later." when trying to save a new flexslider optionset
Comment #19
amaria CreditAttribution: amaria commentedThanks for testing! Yes, I fixed a few bugs a couple days ago but neglected to commit them to my sandbox (although they are committed to the Github repo). In any case, please try the latest code which should fix the bugs you encountered and several more.
Comment #20
bloomt CreditAttribution: bloomt commentedI downloaded the master from https://github.com/coldfrontlabs/flexslider/tree/8.x-2.x, I assume this is the github repo that you are working on. Anyways both of my issues still exist.
Comment #21
amaria CreditAttribution: amaria commentedNo, actually I had to fork that project in order to work on it and my changes to it are pending. You can find the latest here https://github.com/amaria/flexslider/tree/8.x-2.x or in my sandbox.
Comment #22
bloomt CreditAttribution: bloomt commentedI still cannot edit a flexslider field within a view.
Thanks for all the hard work.
Comment #23
amaria CreditAttribution: amaria commentedThe latest version in my sandbox and github should fix the issue with flexslider formatter in views, among other things.
No problem. Thanks for testing!
Comment #24
minorOffense CreditAttribution: minorOffense at Coldfront Labs Inc. commentedI've pulled the latest from your sandbox and pushed it to github and the flexslider project.
If you open pull requests on GitHub I can keep up with merging changes in as they happen and publish them to the flexslider project.
Comment #25
amaria CreditAttribution: amaria commentedOk, I have created another pull request for the latest commit which ports the example module to D8 with the following changes:
- Uses D8 configuration management instead of Features
- Uses an embed View for the "slider with slider nav" example instead of Block with Context placement.
- Adds the FlexSlider example menu to the toolbar for easy access.
- Generates content when installed
- Deletes content and configuration when uninstalled
- Adds more detail to README
So I believe this is pretty much stable enough at least for a dev release. The only thing that doesn't appear to be working at the moment is the debug switch which does not load the non-minified version when set. I'll have to take another look at the library part if this since the Libraries module is not fully functional yet.
Comment #26
amaria CreditAttribution: amaria commentedComment #27
amaria CreditAttribution: amaria commentedAlso added a new commit to fixed a problem with loading the sample files.
Comment #28
bloomt CreditAttribution: bloomt commentedModule is working great for me, it definitely can be pushed to a dev release.
Are you going to build out the integration for Responsive Images?
Comment #29
amaria CreditAttribution: amaria commentedIt's up to the project maintainer to decide if that should be part of this project. I could create a separate project if not.
Comment #30
amaria CreditAttribution: amaria commentedThe D8 dev version is now available for testing.
Comment #31
bloomt CreditAttribution: bloomt commentedComment #32
ecvandenberg CreditAttribution: ecvandenberg as a volunteer commentedJust used flexslider-8.x-2.x-dev (2016-Feb-22) and it works fine in flexslider_fields use. At this moment I think flexslider is the only suitable option I know of for getting slides to work in Drupal 8. Jssor is just not ready and views slideshow is not nice in its responsive behavior. But flexslider is in dev state. So actually we do not have a nice and stable module for slides in Drupal 8.
I guess the dependency with libraries which is also in dev is blocking this module from getting released. Due to the discussion about the need for libraries in Drupal 8 I don't think they come out of dev soon.
Please consider to get rid of the libraries dependency. For the sake of getting a stable 8.x release soon of this very nice module.
Comment #33
minorOffense CreditAttribution: minorOffense at Coldfront Labs Inc. commentedPicture or Responsive images modules can integrate with flex slider on their own (I think picture already does). Otherwise I welcome anyone to create a new contrib module to make it happen. But I won't be adding it to flex slider directly. Too much code to maintain / debug.
Comment #34
darvanenTested using a view and it works well.
Is it the Libraries API dependency that's holding this back from at least being an alpha?
Comment #35
amaria CreditAttribution: amaria commentedFunny you should ask :) I was just writing up a post concerning the roadmap to final release. Will be up shortly.
Comment #36
amaria CreditAttribution: amaria commentedRoadmap here
Comment #37
amaria CreditAttribution: amaria commentedD8 version available since 2016