Skip to content
Snippets Groups Projects
Commit 5ffabc78 authored by Tim van Dijen's avatar Tim van Dijen Committed by Thijs Kinkhorst
Browse files

Fix minor XHTML compliancy issues (#479)

Don't start a <dl> when there's no content for it. Would result in "<dl></dl>".

One error left in: data-clipboard-target="#metadata"
JavaScript should probably be changed to check for id instead of a proprietary attribute.

Add mandatory cols-attribute for non-CSS compatible browsers.

We're one column short
parent dbdddf39
No related branches found
No related tags found
No related merge requests found
......@@ -10,9 +10,8 @@ if ($this->data['isadmin']) {
$this->t('{core:frontpage:login_as_admin}').'</a></p>';
}
$now = time();
echo '<dl>';
if (is_array($this->data['metaentries']['hosted']) && count($this->data['metaentries']['hosted']) > 0) {
echo '<dl>';
foreach ($this->data['metaentries']['hosted'] as $hm) {
echo '<dt>'.$this->t($this->data['mtype'][$hm['metadata-set']]).'</dt>';
echo '<dd>';
......@@ -38,10 +37,11 @@ if (is_array($this->data['metaentries']['hosted']) && count($this->data['metaent
echo '</p></dd>';
}
echo '</dl>';
}
echo '</dl>';
if (is_array($this->data['metaentries']['remote']) && count($this->data['metaentries']['remote']) > 0) {
$now = time();
foreach ($this->data['metaentries']['remote'] as $setkey => $set) {
echo '<fieldset class="fancyfieldset"><legend>'.$this->t($this->data['mtype'][$setkey]).' (Trusted)</legend>';
......
......@@ -163,7 +163,7 @@ if ($this->data['errorcode'] !== null) {
</td>
</tr>
<tr id="mobilesubmit">
<td></td>
<td></td><td></td>
<td>
<button class="btn" tabindex="6">
<?php echo $this->t('{login:login_button}'); ?>
......
......@@ -4,7 +4,7 @@ $this->includeAtTemplateBase('includes/header.php');
<div class="metadatabox">
<button data-clipboard-target="#metadata" id="btncp" class="btn topright" style="margin-right: 0.5em;">
<img src="/<?php echo $this->data['baseurlpath'].'resources/icons/clipboard.svg'; ?>"
alt="Copy to clipboard">
alt="Copy to clipboard" />
</button>
<pre id="metadata">
$metadata['<?php echo $this->data['m']['metadata-index']; unset($this->data['m']['metadata-index']) ?>'] => <?php
......
......@@ -23,7 +23,7 @@ if (isset($this->data['includeTemplate'])) {
<pre id="trackid" class="input-left"><?php echo $this->data['error']['trackId']; ?></pre>
<button data-clipboard-target="#trackid" id="btntrackid" class="btnaddonright">
<img src="/<?php echo $this->data['baseurlpath'].'resources/icons/clipboard.svg'; ?>"
alt="Copy to clipboard">
alt="Copy to clipboard" />
</button>
</div>
</div>
......@@ -54,7 +54,7 @@ if (isset($this->data['errorReportAddress'])) {
<input type="email" size="25" name="email" value="<?php echo htmlspecialchars($this->data['email']); ?>" />
</p>
<p>
<textarea class="metadatabox" name="text" rows="6" style="width: 100%; padding: 0.5em;"><?php
<textarea class="metadatabox" name="text" rows="6" cols="50" style="width: 100%; padding: 0.5em;"><?php
echo $this->t('report_explain'); ?></textarea>
</p>
<p>
......
......@@ -53,9 +53,9 @@
<td style="padding: .3em;"><?php echo $this->t('password'); ?></td>
<td><input id="password" type="password" tabindex="2" name="password" /></td>
</tr>
<tr><td></td><td>
<button type="submit" tabindex="5" id="mobilesubmit" class="btn"><?php echo $this->t('login_button'); ?></button>
</td></tr>
<tr><td></td><td></td><td>
<button type="submit" tabindex="5" id="mobilesubmit" class="btn"><?php echo $this->t('login_button'); ?></button>
</td></tr>
</table>
......
......@@ -11,7 +11,7 @@ $this->includeAtTemplateBase('includes/header.php');
</p>
<p>
<?php echo $this->t('metaconv_selectfile'); ?>
<input type="file" name="xmlfile"/></p>
<input type="file" name="xmlfile" /></p>
<p>
<button type="submit" class="btn"><?php echo $this->t('metaconv_parse'); ?></button>
</p>
......@@ -34,7 +34,7 @@ if (!empty($output)) {
<button data-clipboard-target="#metadata<?php echo $i; ?>" id="btn<?php echo $i; ?>"
class="btn topright" style="margin-right: 0.5em;">
<img src="/<?php echo $this->data['baseurlpath'].'resources/icons/clipboard.svg'; ?>"
alt="Copy to clipboard">
alt="Copy to clipboard" />
</button>
<pre id="metadata<?php echo $i; ?>"><?php
echo htmlspecialchars($text);
......
......@@ -9,7 +9,7 @@ $this->includeAtTemplateBase('includes/header.php'); ?>
<pre id="metadataurl" class="input-left"><?php echo htmlspecialchars($this->data['metaurl']); ?></pre>
<button data-clipboard-target="#metadataurl" id="btnurl" class="btnaddonright">
<img src="/<?php echo $this->data['baseurlpath'].'resources/icons/clipboard.svg'; ?>"
alt="Copy to clipboard">
alt="Copy to clipboard" />
</button>
</div>
<?php } ?>
......@@ -18,7 +18,7 @@ $this->includeAtTemplateBase('includes/header.php'); ?>
<div class="metadatabox">
<button data-clipboard-target="#xmlmetadata" id="btnxml" class="btn topright" style="margin-right: 0.5em;">
<img src="/<?php echo $this->data['baseurlpath'].'resources/icons/clipboard.svg'; ?>"
alt="Copy to clipboard">
alt="Copy to clipboard" />
</button>
<pre id="xmlmetadata"><?php echo $this->data['metadata']; ?></pre>
</div>
......@@ -27,7 +27,7 @@ $this->includeAtTemplateBase('includes/header.php'); ?>
<div class="metadatabox">
<button data-clipboard-target="#phpmetadata" id="btnphp" class="btn topright" style="margin-right: 0.5em;">
<img src="/<?php echo $this->data['baseurlpath'].'resources/icons/clipboard.svg'; ?>"
alt="Copy to clipboard">
alt="Copy to clipboard" />
</button>
<pre id="phpmetadata"><?php echo $this->data['metadataflat']; ?></pre>
</div>
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment