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

Update page title on publish confirmation page #2135

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

mtaylorgds
Copy link
Contributor

What

Update the page title to provide an indication to the user that the page is not just the view page for the manual, but a page for publishing it.

Also remove the unused :page_title value, which was used by the old, legacy, layout but is not needed by the design system one.

Why

To make it easier to distinguish between tabs in the browser, and for consistency with the new discard confirmation page.

Trello

Trello card (this is for the discard page—this PR will bring the publish confirmation page in line with the changes made for the discard confirmation page)

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

Update the page title to provide an indication to the user that the page
 is not just the view page for the manual, but a page for publishing it.

Also remove the unused `:page_title` value, which was used by the old,
 legacy, layout but is not needed by the design system one.
@ryanb-gds ryanb-gds merged commit 39f4e10 into main Aug 23, 2023
@ryanb-gds ryanb-gds deleted the consolidation-remove-unused-page-title-value branch August 23, 2023 09:08
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