infonl / info-techradar

Owner: Edgar. A static site generator for the INFO Technology Radar based on the AOE Tech Radar generator.
https://infonl.github.io/info-techradar/index.html
Other
2 stars 0 forks source link

Chore/use info colors #12

Closed xiduzo closed 1 year ago

xiduzo commented 1 year ago
edgarvonk commented 1 year ago

Screenshot 2022-12-07 at 11 47 36

Nice bright colours. :-) Just checking: is https://wiki.info.nl/display/overall/huisstijl really up to date? I think this may be quite outdated no? Maybe you can check with e.g. Myrte about this?

edgarvonk commented 1 year ago

As far as I know our INFO brand book is on: https://infonl.atlassian.net/wiki/spaces/IHS/pages/19726457/Branding+2022+Files+and+Templates (and then Sharepoint). I think https://wiki.info.nl/display/overall/huisstijl is heavily outdated and needs to be removed. I just sent a Slack msg to Myrte to confirm.

edgarvonk commented 1 year ago

Confirmed by Myrte. https://wiki.info.nl/display/overall/huisstijl is very very old and should not be used. https://infonl.atlassian.net/wiki/spaces/IHS/pages/19726457/Branding+2022+Files+and+Templates is the current one. Can you update the colours @xiduzo accordingly :-)

xiduzo commented 1 year ago

I could change the green color which is used. Unfortunately, the new branding guidelines do not have any other colors for the other segments 🤔

edgarvonk commented 1 year ago

I could change the green color which is used. Unfortunately, the new branding guidelines do not have any other colors for the other segments 🤔

I know.. ;-) That's part of the reason why I did not bother with the INFO branding when creating this tech radar. I'll leave it up to you. Just updating the green might be nice. And you could check with Myrte maybe?

edgarvonk commented 1 year ago

@xiduzo what is the status and your plan for this PR now? Shall we discard it for now?

edgarvonk commented 1 year ago

Decided not to pursue this as the colours are ok like they are and the current INFO styleguide does not fit so well with the radar anyway. Updating deps will be done separately.