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

Fix: Delete files that came with the Design System Starter, that I don't need yet. #7

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

meranicosme
Copy link
Owner

✍️ Description

I made this branch to delete all the files I did not need that came with the Design System Starter repo. I ran npm start and it works. Let me know if there's anything I missed.

Here is my Invision link for the site design. :)

@meranicosme meranicosme changed the title delete all unnecessary files for the project, might create these file… Fix: Delete files that came with the Design System Starter, that I don't need yet. Sep 16, 2019
@import "elements/ol-ul";


// =====================================================================
// 5. Objects
// Class-based selectors which define undecorated design patterns,
// for example media object known from OOCSS.
@import "object/grid";
@import "object/width-limiter";

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think some extra space snuck in here 😄

Copy link
Owner Author

Choose a reason for hiding this comment

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

got it. going to thumbs up stuff I fix

@import "components/skip-to-content";
@import "components/expandable-content";
@import "components/icon";

Copy link
Collaborator

Choose a reason for hiding this comment

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

Lets get rid of that extra space 😄

Copy link
Collaborator

@colbyallenc colbyallenc left a comment

Choose a reason for hiding this comment

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

I noticed some additions made to this PR and i was wondering if they were additions in place of deleted files that came with the design system. Other than that, this pr looks good!!

margin: 0;

Copy link
Collaborator

Choose a reason for hiding this comment

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

a little bit of space snuck in here 😄

Copy link
Owner Author

Choose a reason for hiding this comment

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

this is alright. just made a space for the media query to go after


<!-- <a href="/" class="drizzle-Nav-title">{{data "global.companyName"}} Design System</a>
<!-- <a href="/" class="drizzle-Nav-title">{{data "global.companyName"}} Design System</a>

Copy link
Collaborator

Choose a reason for hiding this comment

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

I feel as though the indentation on this file was good in the original state but I could be wrong! Thoughts, @zastrow ?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Fixed!

might create these files as I go in the project
@colbyallenc colbyallenc merged commit 5f8eb4f into master Sep 18, 2019
@colbyallenc colbyallenc deleted the drizzle-delete-folders branch September 18, 2019 14:04
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.

2 participants