NHSRtheme icon indicating copy to clipboard operation
NHSRtheme copied to clipboard

Should there be more palettes?

Open tomjemmett opened this issue 5 years ago • 3 comments

Currently we have one palette that uses a handful of the colours from the Identity (Blue, Green, Red, Purple, Orange), and then the levels as defined by the Identity (Blues, Neutrals, Support Greens, Highlights). Should we create some more named palettes? Perhaps a RAG palette? Are the existing palettes suitable, or could they be improved?

tomjemmett avatar Mar 12 '19 09:03 tomjemmett

Hi,

Appreciate I am late to the party but this is something I have been looking at recently.

If we refer to the colour palettes in the NHS Identity it seems like the ones in this package do not align properly:

image

Would we be better defining our colour palettes as above ^ instead of the of the colour sections (e.g. Level 1: NHS blues, Level 2: NHS neutrals, etc):

  • core blues: c(DarkBlue = "#003087", BrightBlue = "#0072CE", LightBlue = "#41B6E6", AquaBlue = "#00A9CE")
  • pale neutrals: c(MidGrey = "#768692", PaleGrey = "#E8EDEE")
  • greens: c(DarkGreen = "#006747", Green = "#009639", LightGreen = "#78BE20", AquaGreen = "#00A499")
  • black: c(Black = "#231F20")
  • dark grey: c(DarkGrey = "#425563")
  • highlights: c(Purple = "#330072", DarkPink = "#7C2855", Pink = "#AE2573", DarkRed = "#8A1538", Red = "#DA291C", Orange = "#ED8B00", WarmYellow = "#FFB81C", Yellow = "#FAE100")

Then this would enable you to recreate all of the colour palettes in the identity (core blues + pale neutrals, greens + black + dark grey, highlights) and also custom palettes by combining in a different way.

The default could then be core blues + pale neutrals

ghost avatar May 20 '21 11:05 ghost

Additionally, I am looking to add some functionality like https://github.com/bbc/bbplot/blob/master/R/bbc_style.R into this package if there is appetite?

ghost avatar May 20 '21 11:05 ghost

I think you are probably right - these are much nicer descriptions! If you want to create a PR I will approve and merge :-) Might be worth having both set's of names unless that would cause confusion...

tomjemmett avatar Jun 07 '21 12:06 tomjemmett