I'm no longer implementing any new features into the 7.x branch of this module. However, if there are additional features you want to see in the D8 version of this module. please post your request here.
I want to see D8 get farther into core development before I start working on this branch. But I wanted to get this conversation started. Below are a few things that come to my mind that I'm thinking about changing in the D8 branch:
1. Get rid of the popup option. If you use this feature in the D7 branch, let me know! I haven't heard of anyone that uses this, but I want to keep it around if people do use it.
2. Add hooks that would allow a developer to implement other lightbox solutions, other than Colorbox.
3. Investigate how caching works in D8 to see if there are any ways we can improve the redirect option.
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | splashify-8.x-2.x-1860218-20.patch | 197.83 KB | psend |
Comments
Comment #1
Bastlynn commentedWe're using the popup options. :)
Comment #2
chrisroane commentedUnder the When tab, add a date range option that would specify when the splash page should be active. So for example, you could set the splash to only show once on New Year's Eve. Or you could have the splash page Always show from 7/1/2013 - 7/10/2013 .
I'm also wondering how useful the set of mobile options actually is. It seems to me the most common use case with mobile would be to disable the splash page on mobile devices...and that would be it. If you use this feature (or might end up using) with Splashify, please let me know!
Bastlynn....that is all I needed to hear in order to keep the popup option in the next version. :)
Comment #3
mrfelton commented1. I'd like to see the ability to create multiple splash instances so that is more flexible and can be used differently for different purposes in different parts of the site.
2. Better visibility controls - specifically, I want to be able to apply to specific pages in one specific language.
Comment #4
cllamas commentedI agree with #3, multiple instances with dates would be GREAT.
- mobile popup
-and.. I'm using the popup too! :S
-one cool feature would be able to integrate slideshows to have features like 'what's new' and put few slides.
Comment #5
chrisroane commentedThanks for sharing!
One additional feature that would be useful would be to only display the node and not the full page html. See this issue: https://drupal.org/node/2141503
Comment #6
chrisroane commentedComment #7
nr_stage17dev commentedThings I'd like to see in the D8 version that I've been struggling to manually implement in the D7 version:
- The ability to control delay on splash load from within the module, similar to how this is handled in Popup Onload. In D7, I needed to wrap setTimeout around the colorbox call in splashify_init.js and edit the .inc file to load splashify_init.js instead of splashify_init_min.js to implement this and I haven't gotten around to trying to patch the module.
- Add option to allow the splash to resize dynamically based on content being loaded in ( if this is something that can be implemented better elsewhere, feel free to inform me ).
Comment #8
chrisroane commentedThanks nr_stage17dev....that is a great suggestion.
One feature that came up recently that I think is valuable is the ability to have multiple splash pages on the site with unique settings for each. Not exactly sure how this would work, but I know it would be useful for larger sites.
Also add options for displaying the splash page hourly and monthly.
Add an option to show the splash page during a set time frame.
Comment #9
bburgSeems like in the spirit of the OOP direction in D8, making Splashify use an extendable plugin system would allow developers to have an easier time in creating their own plugins that satisfy their use cases. Lots of the requests I see here could be satisfied in this way without making the module's internal code more complicated.
I am currently trying to investigate how I can use a minipanel for the content of a splash, and it looks like I will need to reimplement splashify as a new module that does this. Much in the same way as https://www.drupal.org/node/2141503
Comment #10
bburgRequest: have the colorbox_options respect the options set in the colorbox module's config.
Comment #11
chrisroane commentedGreat ideas! I've had similar thoughts about making the system more pluggable.
Comment #12
larynAnother feature request -- instead of a full colorbox window, a less intrusive slider that pushes the rest of the page down/up/side and has a narrow strip with the splashify content in it. (For example, you load the page, then the splashify content slides down, maybe in a slightly more pleasing way than the Wikipedia donor appeal that shows up in a narrow strip at the top of the page).
Comment #13
dom. commentedHi there ! Any news on the D8 version ?
Comment #14
chrisroane commentedIt has been pretty crazy, but we do have some code that seems to have a lot of the main features of splashify for D8 integrated. But I need to test it more and do some code cleanup. Thanks for posting here as if there is a big interest I would like to get this released in a few months.
Comment #15
dom. commentedWhy not creating a D8 branch on the module and commit it, so we can help you with it ?
Comment #16
chrisroane commentedYeah, that's a good idea. I just need to find a moment between the large projects we are working on. I'll try to find some time this week do do that. Thanks.
Comment #17
chrisroane commentedSorry for the delay. I tested the code we are using for a client project and it seems to work and have most of the features from the D7 site. I'm going to separated this code from that project and clean up some of the wording and do a quick code review and get this pushed to 8.x. I created a task to get this scheduled. I'm going to try to find time over the next few weeks to do this so we can get some version of the code out there.
The good news is that the way we have this code in the D8 version, we shouldn't conflict with caching. But that will require more testing.
Comment #18
trigdog commentedLooking forward to testing the 8.x version. Thanks!
Comment #19
chrisroane commentedOne of the teams we have working on this project should be posting the code this week. When they post the patch, I will approve it so we can start immediate testing in the Drupal 8 branch.
Comment #20
psend commentedI have already ported a module. Could you please test it and review the code.
Comment #21
chrisroane commentedThanks dd.psend. But I'm getting an error when I try to apply your patch. Can you fix this and post an updated patch file? You can test to see if the patch works by checking out 8.x-2.x branch and applying the patch based on these instructions.
https://www.drupal.org/project/splashify/git-instructions
Let me know if I should try again with a new patch file. Thanks.
Comment #23
chrisroane commentedI resolved the issue by removing the Mobile_Detect file from the patch and manually deleting the file. I committed the code and also just released 8.x-2.x as a version that should end up showing on the main project page.
I need people to test this code and we need to assess what other features we need. There is quite a bit of difference in how this module works compared to the D7 version. One example is that splash pages are organized by groups so you can rotate between different splash pages in certain cases. I also think this module handles caching better than the D7 module did.
Comment #24
karthik_madasamy commentedHi Chrisroance,
Can you please send patch for Start and end date under when tab functionality for Drupal 8.
"Under the When tab, add a date range option that would specify when the splash page should be active. So for example, you could set the splash to only show once on New Year's Eve. Or you could have the splash page Always show from 7/1/2013 - 7/10/2013 .
I'm also wondering how useful the set of mobile options actually is. It seems to me the most common use case with mobile would be to disable the splash page on mobile devices...and that would be it. If you use this feature (or might end up using) with Splashify, please let me know!
Bastlynn....that is all I needed to hear in order to keep the popup option in the next version. :)
"
Comment #25
astonvictor commentedD7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.