I see that E-Mail address and website are displayed on the entries page if they were part of the form. I'd like to see those made into links.
Also, I'm not sure if you guys want patches submitted for this kind of stuff or if we should just tell you about it (including my other post for input type). Since I've modified my version, I could definitely create a patch (or rather a diff, unless you have a read version of your SVN repository accessible) or send you a patched file.
Here is the patch to fix both issues if anyone is interested:
Index: gravityforms.php
===================================================================
--- gravityforms.php (revision 23)
+++ gravityforms.php (working copy)
@@ -2405,6 +2405,14 @@
$value = self::format_date($value, false);// date('Y/m/d \a\t H:i', mysql2date('G', $value));
break;
+ case "email" :
+ $value = "<a href='mailto:" . self::escape_attribute($value) . "' alt='E-Mail " . self::escape_attribute($value) ."' title='E-Mail " . self::escape_attribute($value) . "'>" . self::escape_text($value) . "</a>";
+ break;
+
+ case "website" :
+ $value = "<a href='" . self::escape_attribute($value) . "' alt='Visit " . self::escape_attribute($value) ."' title='Visit " . self::escape_attribute($value) . "'>" . self::escape_text($value) . "</a>";
+ break;
+
default:
$value = self::escape_text($value);
}
@@ -2905,13 +2913,13 @@
switch($field["nameFormat"]){
case "extended" :
- return sprintf("<div class='ginput_complex'><span class='name_prefix'><input name='input_%d.2' id='input_%d.2' value='%s' tabindex='%d' %s/><label for='input_%d.2'>" . __("Prefix", "gravityforms") . "</label></span><span class='name_first'><input name='input_%d.3' id='input_%d.3' value='%s' tabindex='%d' %s/><label for='input_%d.3'>" . __("First", "gravityforms") . "</label></span><span class='name_last'><input name='input_%d.6' id='input_%d.6' value='%s' tabindex='%d' %s/><label for='input_%d.6'>" . __("Last", "gravityforms") . "</label></span><span class='name_suffix'><input name='input_%d.8' id='input_%d.8' value='%s' tabindex='%d' %s/><label for='input_%d.8'>" . __("Suffix", "gravityforms") . "</label></span></div>", $id, $id, $prefix, self::$tab_index++, $disabled_text, $id, $id, $id, $first, self::$tab_index++, $disabled_text, $id, $id, $id, $last, self::$tab_index++, $disabled_text, $id, $id, $id, $suffix, self::$tab_index++, $disabled_text, $id);
+ return sprintf("<div class='ginput_complex'><span class='name_prefix'><input name='input_%d.2' type='text' id='input_%d.2' value='%s' tabindex='%d' %s/><label for='input_%d.2'>" . __("Prefix", "gravityforms") . "</label></span><span class='name_first'><input name='input_%d.3' type='text' id='input_%d.3' value='%s' tabindex='%d' %s/><label for='input_%d.3'>" . __("First", "gravityforms") . "</label></span><span class='name_last'><input name='input_%d.6' type='text' id='input_%d.6' value='%s' tabindex='%d' %s/><label for='input_%d.6'>" . __("Last", "gravityforms") . "</label></span><span class='name_suffix'><input name='input_%d.8' type='text' id='input_%d.8' value='%s' tabindex='%d' %s/><label for='input_%d.8'>" . __("Suffix", "gravityforms") . "</label></span></div>", $id, $id, $prefix, self::$tab_index++, $disabled_text, $id, $id, $id, $first, self::$tab_index++, $disabled_text, $id, $id, $id, $last, self::$tab_index++, $disabled_text, $id, $id, $id, $suffix, self::$tab_index++, $disabled_text, $id);
case "simple" :
return sprintf("<div><input name='input_%d' type='text' value='%s' class='%s' tabindex='%d' %s/></div>", $id, self::escape_attribute($value), self::escape_attribute($size), self::$tab_index++, $disabled_text);
default :
- return sprintf("<div class='ginput_complex'><span class='ginput_left'><input name='input_%d.3' id='input_%d.3' value='%s' tabindex='%d' %s/><label for='input_%d.3'>" . __("First", "gravityforms") . "</label></span><span class='ginput_right'><input name='input_%d.6' id='input_%d.6' value='%s' tabindex='%d' %s/><label for='input_%d.6'>" . __("Last", "gravityforms") . "</label></span></div>", $id, $id, $first, self::$tab_index++, $disabled_text, $id, $id, $id, $last, self::$tab_index++, $disabled_text, $id);
+ return sprintf("<div class='ginput_complex'><span class='ginput_left'><input name='input_%d.3' type='text' id='input_%d.3' value='%s' tabindex='%d' %s/><label for='input_%d.3'>" . __("First", "gravityforms") . "</label></span><span class='ginput_right'><input name='input_%d.6' type='text' id='input_%d.6' value='%s' tabindex='%d' %s/><label for='input_%d.6'>" . __("Last", "gravityforms") . "</label></span></div>", $id, $id, $first, self::$tab_index++, $disabled_text, $id, $id, $id, $last, self::$tab_index++, $disabled_text, $id);
}
case "address" :We plan on making the email and website both linkable in a future release. It is on our to do list so we do plan on making this change.
That's great! Well, I put the patch in my last post, but if you'd rather have it sent by E-Mail or something that's a little cleaner, just let me know.
This will be resolved in the 1.1 release.
