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

add page title to Content Block Manager home page #9800

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

Harriethw
Copy link
Contributor

This sets the beginning of the page title - the
second half, the - GOV.UK Content Block Manager
is set by the shared layout publishing component
[1] so it can't be changed.

Although the page title is Content Block Manager
I've gone with Search as the title, because that
is the main action taken on this page, and it
would be strange to have Content Block Manager
repeated twice.

[1]https://github.com/alphagov/govuk_publishing_components/blob/main/app/views/govuk_publishing_components/components/_layout_for_admin.html.erb

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

Follow these steps if you are doing a Rails upgrade.

This sets the beginning of the page title - the
second half, the ` - GOV.UK Content Block Manager`
is set by the shared layout publishing component
[1] so it can't be changed.

The naming discussed on Slack
https://gds.slack.com/archives/C062Z1FA2ES/p1736500050626609?thread_ts=1736442857.222039&cid=C062Z1FA2ES
as making sense for now - it's what we refer to
the page as in guidance - but we may change later
if more suitable name becomes apparent.

[1]https://github.com/alphagov/govuk_publishing_components/blob/main/app/views/govuk_publishing_components/components/_layout_for_admin.html.erb
@Harriethw Harriethw force-pushed the content-modelling/789-page-title branch from 322d9f0 to 464e98f Compare January 10, 2025 09:52
@Harriethw Harriethw requested a review from pezholio January 10, 2025 10:09
@Harriethw Harriethw marked this pull request as ready for review January 10, 2025 10:52
@Harriethw Harriethw merged commit a1c96f8 into main Jan 10, 2025
19 checks passed
@Harriethw Harriethw deleted the content-modelling/789-page-title branch January 10, 2025 16:02
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