Posted by NancyDru on April 12, 2009 at 11:06pm
5 followers
| Project: | Tabs (jQuery UI tabs) |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
value of attribute "id" invalid: "-" cannot start a name
<div class="drupal-tabs"><ul class="anchors"><li><a href="#-0-1">Newest</a></li><li><a href="#-0-2">Popular</a></li></ul><div id="-0-1" class="fragment"><h2 class="drupal-tabs-title">Newest</h2><div class="panel-pane" id="blog-panel">
Comments
#1
yes, I have the same problem, I am doing my final year project and validation on the page i used panels is not 1.0 xhtml strict
Greg
#2
Panels 2 is no longer available and is unsupported. Marking all Panels 2 issues won't fix.
#3
Actually, this was reported against the wrong version. As of a few seconds ago, Panels 2 is still available and supported on 5.x according to the project page.
#4
In that case, this doesn't contain nearly enough information to do anything about.
That said, I don't think anybody is currently maintaining Panels 2 for Drupal 5, so yes it's still available but I don't think there's anyone to look at the issues. :/
#5
What more information do you need? The generated output is there as well as the text of the message from the HTML Validator. I will try to gather whatever it takes for "nobody" to fix.
#6
Well, to start with, what generated it? I don't recognize the class="fragment" at all. Now that I think about it, it looks like some javascripty tabs code that's screwing up, which Panels doesn't provide at all. Perhaps this is panels tabs module?
But, see, a proper bug report would've told me what kind of panel this is, what panes were in it, and what the settings were, which is why more information is needed. A module maintainer shouldn't have to try to guess these things from context. For one, we'll guess wrong as often as we guess right, and debug the wrong problem.
#7
Yes, we do use the Panel Tabs module, so maybe this should be transferred to that queue.
#8
#9
This is probably an issue in the D6 branch as well.
#10
I'm moving this to the tabs queue as this is where the problem lies.
I have attached a patch that simply trims any leading spaces from the tab_name so they are not replaced with a hyphen.
There is a similar issue with accented characters which points to a filtering process being needed.
#715042: Tab title beginning with an accented character causes an XHTML validation error
#11
changing title.
#12
doh, lets add a patch that can actually be downloaded :)