From d2bbd4f4f4c741dc9af5cf741bec3c7e99f3543e Mon Sep 17 00:00:00 2001
From: Malcolm Butler <32877315+MoJ-Longbeard@users.noreply.github.com>
Date: Thu, 14 Feb 2019 12:12:52 +0000
Subject: [PATCH] Update _summary-list.scss

[PR #1169](https://github.com/alphagov/govuk-frontend/pull/1169) added in some CSS:
- ``word-break: break-word;``, followed by
- ``word-break: break-all;``

``word-break: break-word;`` is the favoured behaviour, but isn't fully supported.
``word-break: break-all;`` is the fallback behaviour, it sometimes results in unfavourable wrapping mid-word.

The order for this CSS needs to be changed to, otherwise the less favourable `break-all` takes precedence.
```CSS
word-break: break-all;
word-break: break-word;
```

**caniuse** said:
> Chrome, Safari and other WebKit/Blink browsers also support the unofficial break-word value

The implication here being that it is not fully supported, so ``word-break: break-all;`` is still needed as a fallback.

Discussed with [Colin Rotherham](https://github.com/colinrotherham)
---
 src/components/summary-list/_summary-list.scss | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/summary-list/_summary-list.scss b/src/components/summary-list/_summary-list.scss
index ad4b25c9a4..a68f9243b7 100644
--- a/src/components/summary-list/_summary-list.scss
+++ b/src/components/summary-list/_summary-list.scss
@@ -58,8 +58,8 @@
   .govuk-summary-list__value {
     // sass-lint:disable no-duplicate-properties
     // Automatic wrapping for unbreakable text (e.g. URLs)
-    word-break: break-word; // WebKit/Blink only
     word-break: break-all; // Standards
+    word-break: break-word; // Chrome/Safari/WebKit/Blink only
   }
 
   .govuk-summary-list__key {