Skip to content

Conversation

@signekb
Copy link
Member

@signekb signekb commented May 23, 2025

Description

This adds a brand.yml and adds some extra examples to test the scss defauls and rules from the brand.

It also does something else very discreetly 🙈

This PR needs an in-depth review.

Checklist

  • Ran just run-all

@signekb signekb requested a review from lwjohnst86 May 23, 2025 11:51
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

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

Only small comments 🎉

from: markdown+emoji
theme:
- brand
- yeti
Copy link
Member

Choose a reason for hiding this comment

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

Is this still necessary? Or it overlaps anything missing from brand?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is where you get the primary colour from. Maybe/probably also other things. I just assumed you would like to keep it. If it's only the colour, I can just add it to the brand and remove yeti?

Copy link
Member

Choose a reason for hiding this comment

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

It's fine ☺️

book:
title: Testing
author: Luke W. Johnston
author: Luke W. Johnston and Signe Kirk Brødbæk
Copy link
Member

Choose a reason for hiding this comment

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

Ha, is this this discreet thing? 😝

Copy link
Member Author

Choose a reason for hiding this comment

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

🙈

Copy link
Member Author

Choose a reason for hiding this comment

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

Very discreet 🤫

Copy link
Member Author

Choose a reason for hiding this comment

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

If you don't mind? 😁

Co-authored-by: Luke W. Johnston <lwjohnst86@users.noreply.github.com>
@signekb signekb requested a review from lwjohnst86 May 23, 2025 12:12
@lwjohnst86 lwjohnst86 merged commit aca0edc into main May 23, 2025
1 check passed
@lwjohnst86 lwjohnst86 deleted the chore/add-brand-yml branch May 23, 2025 12:14
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.

3 participants