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

WIP: Discussion: Separate default and GOV.UK styles #329

Closed
wants to merge 4 commits into from

Conversation

dashouse
Copy link

I was going to do a PR to update the input style from GOV.UK Elements to GOV.UK Frontend but when looking at the multiple uses of this component form an open source perspective I thought it could be useful to break the styles up into "Default", a version of the autocomplete with basic brand-free styling with separate overrides for "GOV.UK Frontend" styles.

There may be a better way to achieve this technically, but I've done the PR as a proof of concept / idea.

Default / generic styles

screen shot 2019-02-18 at 08 52 49

screen shot 2019-02-18 at 08 52 56

GOV.UK overrides (Updated to GOV.UK Frontend size / padding / responsive text)

screen shot 2019-02-18 at 08 54 28

screen shot 2019-02-18 at 08 54 34

@joelanman
Copy link
Contributor

related: #285

@robinwhittleton
Copy link

This would be useful for me, as I’d like to use this on a project I’m working on, but I’m having to override a lot of the default styling.

@aliuk2012 aliuk2012 changed the title Discussion: Separate default and GOV.UK styles DNM: Discussion: Separate default and GOV.UK styles May 1, 2019
@aliuk2012 aliuk2012 changed the title DNM: Discussion: Separate default and GOV.UK styles DO NOT MERGE: Discussion: Separate default and GOV.UK styles May 2, 2019
@aliuk2012 aliuk2012 changed the title DO NOT MERGE: Discussion: Separate default and GOV.UK styles WIP: Discussion: Separate default and GOV.UK styles May 2, 2019
@devansXD
Copy link

Theming would also be useful for several NHS services that I know of. Interested to see this progress.

@NickColley
Copy link
Contributor

NickColley commented Jan 23, 2020

Think this is a good suggestion but closing for now since Dave has left GDS

@NickColley NickColley closed this Jan 23, 2020
@dashouse
Copy link
Author

I merely left the building

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.

5 participants