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

Customise detailed guide pages for Brexit hubs #2097

Merged
merged 4 commits into from
Jun 3, 2021
Merged

Conversation

hannako
Copy link
Contributor

@hannako hannako commented May 4, 2021

What

We are using the detailed guide doc type to publish the brexit child taxon pages, but we need to customise a few things.

See commit message for more details.

The review apps for this pr are pointed at integration where the detailed guides have been published.

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️
https://trello.com/c/7bevQJP4/1497-investigate-options-to-hide-display-components-on-a-detailed-guide-page
https://trello.com/c/TaSxtjir/1540-build-custom-templates-for-brexit-child-taxon-pages

@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-98ffph May 4, 2021 17:46 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 2df4824 to be8238e Compare May 4, 2021 17:48
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-98ffph May 4, 2021 17:48 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 69f26b4 to be333d4 Compare May 19, 2021 09:13
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-isn9cx May 19, 2021 09:14 Inactive
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 19, 2021 09:23 Inactive
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 19, 2021 13:03 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 937252c to 157a6a6 Compare May 19, 2021 13:30
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 19, 2021 13:31 Inactive
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 19, 2021 15:12 Inactive
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 20, 2021 09:58 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 4644da1 to 10284cb Compare May 20, 2021 10:01
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 20, 2021 10:01 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 10284cb to 0611010 Compare May 20, 2021 10:07
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 20, 2021 10:07 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 0611010 to 3577402 Compare May 20, 2021 10:24
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 20, 2021 10:25 Inactive
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 21, 2021 10:56 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 784cc5f to c1dae30 Compare May 24, 2021 07:50
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 24, 2021 07:50 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from c1dae30 to 1e9c046 Compare May 27, 2021 15:18
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 15:19 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 1e9c046 to 50b89bb Compare May 27, 2021 15:26
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 15:26 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 50b89bb to 8536e46 Compare May 27, 2021 15:34
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 15:34 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 8536e46 to 323713e Compare May 27, 2021 19:37
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 19:37 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 323713e to 9dcdda1 Compare May 27, 2021 19:52
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 19:52 Inactive
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 20:27 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 18d883d to 5c02b04 Compare May 27, 2021 20:30
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 20:31 Inactive
hannako added 2 commits May 27, 2021 21:32
Further modifications
- Hide publisher metadata and print link
- Render description field as a link
- Hide print link at bottom of page
- Hide the context field ("Guidance") from above the title
- Replace the superbreadcrumb with a standard breadcrumb.
- Reduce description size and hide published date
@hannako hannako force-pushed the brexit_hub_spike branch from 5c02b04 to f634030 Compare May 27, 2021 20:33
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 20:33 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from f634030 to 3371176 Compare May 27, 2021 20:37
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 20:37 Inactive
@hannako hannako force-pushed the brexit_hub_spike branch from 3371176 to 7a43806 Compare May 27, 2021 20:41
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 27, 2021 20:41 Inactive
@@ -42,6 +42,8 @@
<% unless @do_not_show_breadcrumbs %>
<% if @content_item.try(:back_link) %>
<%= render 'govuk_publishing_components/components/back_link', href: @content_item.back_link %>
<% elsif @content_item.brexit_hub_page? %>
<%= render 'govuk_publishing_components/components/breadcrumbs', breadcrumbs: [ { url: "/", title: "Home" } , { url: "/transition", title: "Brexit" } ] %>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once we have updated the contextual nav in publishing components this hardcoding can be removed.

@hannako hannako marked this pull request as ready for review May 27, 2021 20:42
@hannako hannako requested a review from sihugh May 28, 2021 08:33
@hannako hannako changed the title DO NOT MERGE: spike changes to detailed guide page for brexit hubs DO NOT MERGE: customised detailed guide pages for brexit hubs May 28, 2021
@hannako hannako changed the title DO NOT MERGE: customised detailed guide pages for brexit hubs DO NOT MERGE: customise detailed guide pages for brexit hubs May 28, 2021
Copy link
Contributor

@sihugh sihugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice :-)

I wonder if we could go a bit further in the helper, and maybe use the locale file so that we can avoid putting content in the view, and isolate the changes a bit further?

app/views/content_items/detailed_guide.html.erb Outdated Show resolved Hide resolved
- english only at this point
- simplify brexit_hub_page presenter
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-pxjxwy May 28, 2021 19:29 Inactive
@hannako hannako changed the title DO NOT MERGE: customise detailed guide pages for brexit hubs Customise detailed guide pages for Brexit hubs May 28, 2021
@bevanloon bevanloon temporarily deployed to government-f-brexit-hub-0kat3g June 3, 2021 08:49 Inactive
@sihugh sihugh merged commit 0aac6bc into main Jun 3, 2021
@sihugh sihugh deleted the brexit_hub_spike branch June 3, 2021 08:51
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