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 dataset page markup for the timeseries partial #833

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

owenatgov
Copy link
Contributor

What

Update the markup, classnames and styling on the individual dataset view timeseries partial.

Why

DGU Find is currently using govuk elements, an outdated implementation of the govuk design system. This PR is one of several that attempts to slowly remove the elements implementation and replace it with up to date implementations from govuk frontend and govuk publishing components.

This PR was created as part of an effort to break down the existing dataset view PR which had grown too large to safely review in one go as it contains a large number of individual features.

Visual changes

TBC

Card: https://trello.com/c/h0XbWxq8/233-update-markup-on-find-dataset-page

@bevanloon bevanloon temporarily deployed to datagovuk-fi-dataset-pa-qmfvv9 October 6, 2020 14:33 Inactive
@bevanloon bevanloon temporarily deployed to datagovuk-fi-dataset-pa-qmfvv9 October 6, 2020 15:54 Inactive
@owenatgov owenatgov force-pushed the dataset-page-timeseries-data branch from 1ee15a1 to c818fc7 Compare October 6, 2020 16:18
@bevanloon bevanloon temporarily deployed to datagovuk-fi-dataset-pa-qmfvv9 October 6, 2020 16:19 Inactive
@owenatgov owenatgov force-pushed the dataset-page-timeseries-data branch from c818fc7 to 0e2329e Compare October 6, 2020 16:56
@bevanloon bevanloon temporarily deployed to datagovuk-fi-dataset-pa-qmfvv9 October 6, 2020 16:57 Inactive
@owenatgov owenatgov force-pushed the dataset-page-timeseries-data branch from 0e2329e to 7277205 Compare December 8, 2020 17:21
@bevanloon bevanloon temporarily deployed to datagovuk-fi-dataset-pa-qmfvv9 December 8, 2020 17:22 Inactive
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