If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[newsletter] /existing page styling is off

RESOLVED FIXED

Status

www.mozilla.org
Newsletters
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Jessilyn Davis, Assigned: dpoirier)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Created attachment 774828 [details]
Screenshot

See attachement.

Looks like the styling sheet on /newsletter/existing is messed up - the descriptions are all bolded and the newsletter titles are small.
(Assignee)

Comment 1

4 years ago
Opened PR https://github.com/mozilla/bedrock/pull/1072 and asked sgarrity to review.
Status: NEW → ASSIGNED
OS: Mac OS X → All
Hardware: x86 → All

Comment 2

4 years ago
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/7ec8b7aba21cf0bbb154b790786684cf7c126d91
Bug 893144 - /newsletter/existing/ styling

The styling sheet on /newsletter/existing ws messed up - the
descriptions were all bolded and the newsletter titles were small.

Reverted styling of the TH part of the table to what it was in an
earlier version of newsletter.less. No way to tell what happened
to it due to rebasing on the original branch.

https://github.com/mozilla/bedrock/commit/ea5ae0256de9ee75d94c5b1923d53c6d95b71775
Merge pull request #1072 from dpoirier/bug-893144-newsletter-existing-styling

Bug 893144 - /newsletter/existing/ styling
I'll check this when it goes out to prod
(Reporter)

Comment 4

4 years ago
Thanks Raymond! If you can check that the page still works for unsubscribing, and re-subscribing, updating language, etc. - that'd be spectacular! :D

(Ie change subscriptions for a subscriber, and when you go back to their email preference center (/en-US/) it should show your changes)
(Reporter)

Comment 5

4 years ago
This went live - just stumbled upon it and gave it a whirl.

It looks good to me :)
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.