Visitar URL original
add design system files by fzhao99 · Pull Request #3 · CDCgov/dibbs-design-system · GitHub
Skip to content

add design system files - #3

Merged
fzhao99 merged 13 commits into
mainfrom
bob/add-design-system-styles
Jun 12, 2025
Merged

fzhao99 merged 13 commits into
mainfrom
bob/add-design-system-styles

Conversation

@fzhao99

@fzhao99 fzhao99 commented Jun 11, 2025 •

Copy link
Copy Markdown
Contributor

Initial setup for the design system overrides, including a writeup for the repo itself

Screenshot 2025-06-11 at 2 59 49 PM Screenshot 2025-06-11 at 2 59 55 PM

Comment thread README.md Resolved
Comment thread sass/_variables.scss Outdated
Comment on lines +8 to +14
$query-connector-max-width: 45rem;
$query-connector-wide-max-width: 90rem;
$query-connector-content-margins: 13.75rem;
$query-connector-content-margins-wide: 10rem;
$query-connector-content-width-wide: calc(
$query-connector-wide-max-width - 2 * $query-connector-content-margins-wide
);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should these still be in here? or renamed to dibbs or similar?

@fzhao99 fzhao99 Jun 11, 2025 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah yes, will ask the designers if these are system level defaults or should be something set at the project level. Changed the naming for now

Comment thread sass/_variables.scss Outdated
Comment on lines +8 to +14
$query-connector-max-width: 45rem;
$query-connector-wide-max-width: 90rem;
$query-connector-content-margins: 13.75rem;
$query-connector-content-margins-wide: 10rem;
$query-connector-content-width-wide: calc(
$query-connector-wide-max-width - 2 * $query-connector-content-margins-wide
);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should we change $query-connector-* to something more generic? Maybe $app-* or $dibbs-app-*?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

jinx

Comment thread sass/_variables.scss Resolved
Comment thread sass/_variables.scss Resolved
Comment thread sass/design_system.scss Resolved
@fzhao99 fzhao99 changed the title design system files add design system files Jun 11, 2025
Comment thread sass/_variables.scss Resolved
Comment thread sass/design_system.scss Resolved
Comment thread sass/design_system.scss Resolved
Comment thread README.md Resolved

@mcmcgrath13 mcmcgrath13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

a couple minor nits, but as this is a living-starting-place-document, I think we can go ahead and merge and continue to iterate as we implement and learn

thanks for pulling this together!

@kevinfiol kevinfiol left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

To echo @mcmcgrath13 , I'm comfortable with moving forward with this in its current state and iterating as we implement the styles. Thanks for your work on this!

@fzhao99
fzhao99 merged commit d08a718 into main Jun 12, 2025
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.

4 participants