Bug 1524632 Comment 12 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

## Beta/Release Uplift Approval Request

### Feature/Bug causing the regression

Bug 1524632

### User impact if declined

Less optimal visual layout for Discovery Stream

### Is this code covered by automated tests?

No

### Has the fix been verified in Nightly?

Yes

### Needs manual test from QE?

Yes

### If yes, steps to reproduce

1. Open `about:config`
2. Change `browser.newtabpage.activity-stream.discoverystream.config` to `{"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}`
3. Open a new tab
4. Confirm that layout doesn't look broken

### List of other uplifts needed

None

### Risk to taking this patch

Low

### Why is the change risky/not risky? (and alternatives if risky)

CSS only patch aside from 1 minor HTML tweak (class name added, truncation removed on `content-src/components/DiscoveryStreamComponents/Hero/Hero.jsx`)

### String changes made/needed
## Beta/Release Uplift Approval Request

### Feature/Bug causing the regression

Bug 1524632

### User impact if declined

Less optimal visual layout for Discovery Stream

### Is this code covered by automated tests?

No

### Has the fix been verified in Nightly?

No

### Needs manual test from QE?

Yes

### If yes, steps to reproduce

1. Open `about:config`
2. Change `browser.newtabpage.activity-stream.discoverystream.config` to `{"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}`
3. Open a new tab
4. Confirm that layout doesn't look broken

### List of other uplifts needed

None

### Risk to taking this patch

Low

### Why is the change risky/not risky? (and alternatives if risky)

CSS only patch aside from 1 minor HTML tweak (class name added, truncation removed on `content-src/components/DiscoveryStreamComponents/Hero/Hero.jsx`)

### String changes made/needed
## Beta/Release Uplift Approval Request

### Feature/Bug causing the regression

Bug 1524632

### User impact if declined

Less optimal visual layout for Discovery Stream

### Is this code covered by automated tests?

No

### Has the fix been verified in Nightly?

No

### Needs manual test from QE?

Yes

### If yes, steps to reproduce

1. Open `about:config`
2. Change `browser.newtabpage.activity-stream.discoverystream.config` to `{"enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=40249-e88c401e1b1f2242d9e441c4&layout_variant=dev-test-all"}`
3. Open a new tab
4. Confirm that layout doesn't look broken

### List of other uplifts needed

Bug 1524321

### Risk to taking this patch

Low

### Why is the change risky/not risky? (and alternatives if risky)

CSS only patch aside from 1 minor HTML tweak (class name added, truncation removed on `content-src/components/DiscoveryStreamComponents/Hero/Hero.jsx`)

### String changes made/needed

Back to Bug 1524632 Comment 12