[maemo-community] [maemo2midgard] Breadcrumbs problem

[From: Henri Bergius]

Niels Breet wrote:
> Hi,

Hi!

> I noticed that the breadcrumbs aren't links in Firefox and Opera on the
> 770, because the href uses single quotes instead of double qoutes. I don't
> have access to the breadcrumb code, so that is why I mention in here.

Single quotes are actually valid for XHTML...
http://lists.xml.org/archives/xml-dev/200402/msg00204.html

In any case, I've modified the breadcrumb helper code in Midgard SVN:

--- midcom.core/midcom/helper/nav.php   (revision 4899)
+++ midcom.core/midcom/helper/nav.php   (working copy)
@@ -515,7 +515,7 @@
             {
                 if ($current_class !== null)
                 {
-                    $result .= "<span
class='{$current_class}'>{$data[MIDCOM_NAV_NAME]}</span>";
+                    $result .= "<span
class=\"{$current_class}\">{$data[MIDCOM_NAV_NAME]}</span>";
                 }
                 else
                 {
@@ -524,8 +524,8 @@
             }
             else
             {
-                $result .= "<a href='{$data[MIDCOM_NAV_URL]}'"
-                  . (is_null($class) ? '' : " class='{$class}'")
+                $result .= "<a href=\"{$data[MIDCOM_NAV_URL]}\""
+                  . (is_null($class) ? '' : " class=\"{$class}\"")
                   . ">{$data[MIDCOM_NAV_NAME]}</a>{$separator}";
             }
         }


> - Niels

/Henri