Changeset 1339025

Timestamp:

01/29/2016 04:23:09 PM (10 years ago)

Author:

pagefrog

Message:

adding assets for reals

Location:

pagefrog/trunk

Files:

• README.txt (modified) (1 diff)
• admin/class-pagefrog-instant-articles-parser.php (modified) (1 diff)
• admin/pagefrog-amp-template-styles.php (added)
• assets (added)
• assets/banner-1544x500.jpg (added)
• assets/banner-772x250.jpg (added)
• assets/icon-128x128.jpg (added)
• assets/icon-256x256.jpg (added)
• assets/screenshot-1.png (added)
• assets/screenshot-2.png (added)
• assets/screenshot-3.png (added)
• assets/screenshot-4.png (added)

pagefrog/trunk/README.txt

@@ -82 +82 @@
 = 1.0.3.1 =
 *Release Date - January 29, 2016*
+
 * Added screenshots, icon, banner images
+* Various AMP & Instant Articles fixes
 
 = 1.0.3 =
 *Release Date - January 28, 2016*
+
 * Launch release
 * Facebook Instant Articles Support (beta)

pagefrog/trunk/admin/class-pagefrog-instant-articles-parser.php

@@ -686 +686 @@
         }
 
+        // ensure <a> tag href attributes aren't blank - set to '#' if blank
+        for ($index = 0; $index < $node->childNodes->length; $index += 1) {
+            $child = $node->childNodes->item($index);
+            if ($child->nodeName == 'a') {
+                if ($child->attributes == NULL ||
+                    $child->attributes->getNamedItem("href") == NULL ||
+                    trim($child->attributes->getNamedItem("href")->value) == "") {
+                        $child->setAttribute("href", "#");
+                }
+            }
+        }
+
         // remove sequential <br> tags when more than 3 (at most 2 should be present)
         $prev_br_children = 0;