Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tidy up account layout template for DI launch #2380

Merged
merged 2 commits into from
Oct 27, 2021

Conversation

danacotoran
Copy link
Contributor

@danacotoran danacotoran commented Oct 25, 2021

What

Implement small updates to the account layout template to ensure a smooth transition as we launch with DI:

  • remove the security link from the account sidebar
  • change the phase banner to "Beta", not "Alpha"
  • remove grey feedback footer box

Visual Changes

Before

Screenshot 2021-10-25 at 17 23 28

After

Screenshot 2021-10-25 at 17 21 37


https://trello.com/c/vIjbnGSj/1087-account-pages-tidy-up-for-launch

Related PR in static updates the layout header logo/link and removes the additional links from the footer in favour of the more stripped down version: alphagov/static#2623

@govuk-ci govuk-ci temporarily deployed to govuk-publis-update-acc-w0iylh October 25, 2021 09:44 Inactive
@govuk-ci govuk-ci temporarily deployed to govuk-publis-update-acc-w0iylh October 25, 2021 09:45 Inactive
@danacotoran danacotoran force-pushed the update-account-sidenav branch from 6ae94b3 to 5f466fa Compare October 25, 2021 16:18
@danacotoran danacotoran changed the title Remove Security and Privacy link from account nav Tidy up account layout template for DI launch Oct 25, 2021
@govuk-ci govuk-ci temporarily deployed to govuk-publis-update-acc-w0iylh October 25, 2021 16:18 Inactive
@danacotoran danacotoran requested a review from barrucadu October 25, 2021 16:24
Remove the security link from the account sidebar as we are launching
without that for now with DI.
@danacotoran danacotoran force-pushed the update-account-sidenav branch from 5f466fa to 26baec4 Compare October 27, 2021 08:57
@govuk-ci govuk-ci temporarily deployed to govuk-publis-update-acc-w0iylh October 27, 2021 08:57 Inactive
- change the phase banner to "Beta", not "Alpha"
- remove grey feedback footer box
@danacotoran danacotoran force-pushed the update-account-sidenav branch from 26baec4 to 310eb84 Compare October 27, 2021 09:21
@govuk-ci govuk-ci temporarily deployed to govuk-publis-update-acc-w0iylh October 27, 2021 09:21 Inactive
@danacotoran danacotoran merged commit 8fe4273 into master Oct 27, 2021
@danacotoran danacotoran deleted the update-account-sidenav branch October 27, 2021 09:37
@danacotoran danacotoran mentioned this pull request Oct 27, 2021
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.

3 participants