Hi,

I've install the module, turn "Use AJAX:" to "yes", but when I try to set "Use Ajax + Fade:" to "yes", set a duration, and validate, "Use Ajax + Fade:" kept stuck on "No".
When I retry to configure it, its still on "yes" and the duration is still here too, but validating make nothing.

And of course, the Fade effect does'nt work.

Any idea to fix this ?

Thanks

CommentFileSizeAuthor
#12 views_ajax_fade.patch1.72 KBdoubleff

Comments

alpp’s picture

i'm having the same problem.

godot882’s picture

same here...

tlattimore’s picture

Assigned: Unassigned » tlattimore

Sorry about this issue. I will look into it.

Could you all list what version of Drupal and Views you are running?

fagost84’s picture

same here...

Drupal core 7.19
views 7.x-3.5
views_ajax_fade 7.x-1.2

b.one’s picture

drupal core 7.20
views 7.x-3.5
views_ajax_fade 7.x-1.2

alpp’s picture

drupal core 7.21
views 7.x-3.5
views_ajax_fade 7.x-1.2

eboss’s picture

Any progress with this issue?

tlattimore’s picture

I really don't know what is going on here. I would say it is a fringe issue, but with the number of people that have reported this issue I just don't know..

I am unable to replicate it in my testing with views-7.x-3.5 and Drupal-7.20 as other users reported.

b.one’s picture

maybe it's due to another module, maybe we could list modules used on our sites ?

tlattimore’s picture

If you are using other views plugins that may cause an issue.

b.one’s picture

i'm using Views Fluid Grid 7.x-3.0

doubleff’s picture

StatusFileSize
new1.72 KB

I had the same issue with a site translated in french ...
A simple modifications to have it working now, by translating the Yes parameter :

in file views_ajax_fade.module :
@@ -33,7 +33,7 @@ function views_ajax_fade_views_ajax_data

   // Replace Views' method replaceWith with our custom command
   // to replace the content with a fade. See: views_ajax_fade.js.
-  if ($use_fade == "Yes") {
+  if ($use_fade == t("Yes")) {
     foreach ($commands as &$command) {
       if ($command['method'] == 'replaceWith') {
         $command['command'] = 'pagerViewFade';

and in file views_plugin_ajax_fade.inc :
@@ -99,7 +99,7 @@ class views_plugin_ajax_fade extends vie

function options_summary(&$categories, &$options) {
     $use_fade = check_plain(trim($this->display->get_option('vaf_use_fade')));
     $duration = check_plain(trim($this->display->get_option('vaf_fade_duration')));
-    if ($use_fade == 'Yes') {
+    if ($use_fade == t('Yes')) {
       $use_fade = t('Yes') . ' (' . $duration . 'ms)';
     }
     else {

Hope it helps ...

steeph’s picture

Status: Active » Reviewed & tested by the community

Yep, it works only if the site is English. The patch fixed it.

tlattimore’s picture

Status: Reviewed & tested by the community » Needs work

Sorry for the long delay in giving this issue attention. I somehow missed these latest posts and that patch.

+++ views_ajax_fade/views_ajax_fade.module	2013-04-30 14:05:52.000000000 +0200
@@ -33,7 +33,7 @@ function views_ajax_fade_views_ajax_data
+  if ($use_fade == t("Yes")) {

Somehow it feels like calling t() within a control operator just efficient - or rather I have approached this completely wrong. Maybe look for another value to check against? like 0/1 for the form element selected? If we can't figure out a solution there - I am open to submitted committing this patch.

Thanks so much for the patch, doubleff! Hope we can get this solved asap.

tlattimore’s picture

Assigned: tlattimore » Unassigned

I can't think of a better way to do this at this point and want this issue resolved. So I have committed it to 7.x-1.x. The change will be included in the next release.

http://drupalcode.org/project/views_ajax_fade.git/commit/27d7724a4492818...

tlattimore’s picture

Status: Needs work » Fixed

this is fixed.

Status: Fixed » Closed (fixed)

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