Skip to content

Update Invalid html syntax and remove IE8 support from hmpo support - #199

Open
aliuk2012 wants to merge 3 commits into
HMPO:masterfrom
Laings-Online-Limited:update-invalid-html-syntax
Open

aliuk2012 wants to merge 3 commits into
HMPO:masterfrom
Laings-Online-Limited:update-invalid-html-syntax

Conversation

@aliuk2012

Copy link
Copy Markdown

Best to review each commit separately.

Addresses a couple issues raised by W3C HTML Validator and removes mentions of IE8 and support for it as GOV.UK Frontend v5+ does not support IE let alone IE8 https://github.com/alphagov/govuk-frontend/blob/main/docs/contributing/browser-support.md#how-we-provide-support-for-different-browsers

W3C HTML Validator raised the following issue

```
Error: Bad value true for attribute novalidate on element form.
```

This is because this component was rendering

```
<form novalidate="true"...>
```

When the newer syntax for boolean attributes is to reference the attribute alone

```
<form novalidate .....>
```

Signed-off-by: Alistair Laing <aliuk2012@users.noreply.github.com>
…nd hmpoTemplate components

W3C HTML Validator raised the following information messgae about self-closing tags using trailing slash

```
 Trailing slash on void elements has no effect and interacts badly with unquoted attribute values.

 ```

 instead of

 ```
 <link href="...." />
 <input type="text" />
 ```

 the markup should be

 ```
 <link href="...." >
 <input type="text" >
 ```

Signed-off-by: Alistair Laing <aliuk2012@users.noreply.github.com>
IE8 is no longer supported by govuk-frontend. I also searched references to `IE8`,  `applicationStyleSheetIE8`, `applicationStyleSheet` and this was the only file that referenced IE8

Signed-off-by: Alistair Laing <aliuk2012@users.noreply.github.com>
@aliuk2012

Copy link
Copy Markdown
Author

Any update on when this might be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants