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

Guide print stylesheet: from gov.uk text duplicated #626

Merged
merged 1 commit into from
Jun 26, 2014

Conversation

camillebaldock
Copy link
Contributor

This fixes this issue:
https://govuk.zendesk.com/agent/#/tickets/764799
When printing out a guide, the title is followed by "a guide from GOV.UK from GOV.UK"
This change fixes that duplication.
I will be opening a new pivotal ticket to track the issue that "from GOV.UK" is being added to the print copy even for Welsh print-outs: this is a less straightforward change as it ties in to another discussion on locales on HTML pages.

@camillebaldock
Copy link
Contributor Author

" from GOV.UK" is currently being added at the end of all print titles as part of the core print CSS in static

@dsingleton
Copy link
Contributor

This looks good to me, but it might be worth getting a native welsh speaker in the office (Rhiannon) to double check the new strings make sense?

@camillebaldock camillebaldock changed the title Guide print stylesheet: from gov.uk text duplicated [do-not-merge]Guide print stylesheet: from gov.uk text duplicated Jun 25, 2014
@camillebaldock
Copy link
Contributor Author

DO NOT MERGE: currently undergoing product discussion over whether static should be adding "from GOV.UK" or not to the end of title

@dsingleton
Copy link
Contributor

I don't think we need product discussion on this change, as it's just removing the duplication in guides? I would like to get that second set of eyes from a native welsh speaker tho. Assuming that's fine then consider this a pre-emptive LGTM and merged when i18n approved.

@camillebaldock
Copy link
Contributor Author

Product discussion with Roo Reynolds has concluded that we should remove both "a guide from gov.uk" from the print template, and stop adding "from gov.uk" to the print stylesheet (alphagov/static#428)

@camillebaldock camillebaldock changed the title [do-not-merge]Guide print stylesheet: from gov.uk text duplicated Guide print stylesheet: from gov.uk text duplicated Jun 26, 2014
QuantumCatgirl pushed a commit that referenced this pull request Jun 26, 2014
Guide print stylesheet: from gov.uk text duplicated

This fixes this issue:
https://govuk.zendesk.com/agent/#/tickets/764799
When printing out a guide, the title is followed by "a guide from GOV.UK from GOV.UK"
This change fixes that duplication. 
I will be opening a new pivotal ticket to track the issue that "from GOV.UK" is being added to the print copy even for Welsh print-outs: this is a less straightforward change as it ties in to another discussion on locales on HTML pages.
@QuantumCatgirl QuantumCatgirl merged commit 4730fee into master Jun 26, 2014
@QuantumCatgirl QuantumCatgirl deleted the print-guide-from-govuk-dup branch June 26, 2014 10:22
QuantumCatgirl pushed a commit to alphagov/static that referenced this pull request Jun 26, 2014
Remove "from gov.uk" on print stylesheet

This fix stems from this issue: 
https://govuk.zendesk.com/agent/#/tickets/764799
which created inconsistencies like"
https://www.gov.uk/lwfans-ceisio-gwaith/print (language inconsistency)
and
https://www.gov.uk/rent-room-in-your-home/print (duplication!)
Roo Reynolds made the decision to remove both "a guide from gov.uk" from the print template, and stop adding "from gov.uk" to the print stylesheet: this fix together with a fix to frontend (alphagov/frontend#626) addresses that.
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.

4 participants