Esri / calcite-maps

A Bootstrap theme for designing, styling and creating modern map apps.
http://esri.github.io/calcite-maps/samples/index.html
Apache License 2.0
239 stars 173 forks source link

ESRI Leaflet Demo Needs W3C Validation #120

Open pmacMaps opened 8 years ago

pmacMaps commented 8 years ago

Can the ESRI Leaflet demo HTML file be updated to pass W3C validation?

Whenever I paste the file into the validator, I get the following errors:

  1. Error: Element div is missing one or more of the following attributes: role. From line 75, column 4; to line 75, column 76 se -->↩ <div id="mainNav" class="collapse navbar-collapse" aria-expanded="false">↩ < Attributes for element div: Global attributes
  2. Error: Attribute aria-lablledby not allowed on element ul at this point. From line 82, column 7; to line 82, column 63 /a>↩ <ul class="dropdown-menu" aria-lablledby="optionsToggle">
    Attributes for element ul: Global attributes
  3. Error: No space between attributes. At line 90, column 32 ="settingsMenu"class="" role="
  4. Error: The aria-controls attribute must point to an element in the same document. From line 58, column 5; to line 58, column 149 der">↩ <button type="button" class="navbar-toggle collapsed" data-toggle="collapse" data-target="#mainNav" aria-expanded="false" aria-controls="navbar">
  5. Error: The aria-labelledby attribute must point to an element in the same document. From line 100, column 2; to line 100, column 101 dow -->↩↩ <div class="modal fade" id="modalSplash" tabindex="-1" role="dialog" aria-labelledby="splashlModal">
alaframboise commented 8 years ago

@pmacMaps thanks. Some of that just didn't make the clean up :) but I'll take a look at this and get it dialed in. Thanks.