Skip to content

Commit

Permalink
fix(multi-file-upload): remove unsemantic HTML
Browse files Browse the repository at this point in the history
Use `<div>` instead of some lone `<dd>`s. This isn't
semantic HTML so we should stop pretending it is.
  • Loading branch information
gregtyler committed Apr 25, 2022
1 parent f382f22 commit ed0f09c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/moj/components/multi-file-upload/multi-file-upload.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,11 +99,11 @@ if(MOJFrontend.dragAndDropSupported() && MOJFrontend.formDataSupported() && MOJF
MOJFrontend.MultiFileUpload.prototype.getFileRowHtml = function(file) {
var html = '';
html += '<div class="govuk-summary-list__row moj-multi-file-upload__row">';
html += ' <dd class="govuk-summary-list__value moj-multi-file-upload__message">';
html += ' <div class="govuk-summary-list__value moj-multi-file-upload__message">';
html += '<span class="moj-multi-file-upload__filename">'+file.name+'</span>';
html += '<span class="moj-multi-file-upload__progress">0%</span>';
html += ' </dd>';
html += ' <dd class="govuk-summary-list__actions moj-multi-file-upload__actions"></dd>';
html += ' </div>';
html += ' <div class="govuk-summary-list__actions moj-multi-file-upload__actions"></div>';
html += '</div>';
return html;
};
Expand Down

0 comments on commit ed0f09c

Please sign in to comment.