Skip to content

Conversation

@scottkellum
Copy link
Member

Bring Modular Scale up to current Sass standards.

@scottkellum
Copy link
Member Author

@robsonsobral
Copy link
Collaborator

Hi, Scott!

Why not to move the _modularscale.scss to modularscale/_index.scss? The @use stays the same, but the files are kept all within a single folder.

I'm sorry for bother you twice in a day.

@scottkellum
Copy link
Member Author

@robsonsobral Not a bother! Happy to get feedback!

@robsonsobral
Copy link
Collaborator

@scottkellum ? Have you given it up?

@scottkellum
Copy link
Member Author

@robsonsobral Ha yeah I have no time for this for the next few months 😅🙃

@robsonsobral
Copy link
Collaborator

I think we have a good thing here, @scottkellum . Let's give it some time?

@scottkellum
Copy link
Member Author

Just want to let you know I see this and am trying to figure out how to allocate time to wrap this up.

@Lukortech
Copy link
Collaborator

I wanted to bump it a bit as I have some deprecation warnings while using this fantastic tool.
If there's any way I could help out let me know @scottkellum

@robsonsobral
Copy link
Collaborator

@Lukortech , I'm so sorry for letting you (and @scottkellum ) down.

I saw your message and planned to use a Brazilian holiday last week to take care of things, but I got sick.

Can you give some time to update the code?

@scottkellum
Copy link
Member Author

I feel that if I wade into this, I will only be a bottleneck. I’ve been drifting further and further away from this project. @robsonsobral and @Lukortech, feel free to merge PRs and push changes as you see fit. A code review might be wanted, so @Lukortech, maybe you can offer that when @robsonsobral is ready?

@robsonsobral
Copy link
Collaborator

I've updated the syntax, @Lukortech ! Now, we're using the latest versions of SASS and True! Now, I need to review things, specially the related to @scottkellum 's Typetura, as I don't know it.

@scottkellum
Copy link
Member Author

Typetura stuff should probably be removed at this point.

* Terminal: `npm install modularscale-sass --save-dev`
* SCSS: `@import 'modularscale'`
- Terminal: `npm install modularscale-sass --save-dev`
- SCSS: `@use '~modularscale-sass/stylesheets/modularscale' as ms;`
Copy link
Collaborator

Choose a reason for hiding this comment

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

The long awaited change 🚀

@scottkellum
Copy link
Member Author

@Lukortech Thank you for your review. I added you as a maintainer if you wish to take a more active role.

@robsonsobral are we ready to go?

@robsonsobral
Copy link
Collaborator

@scottkellum , I opened 2 issues that need fixing, and I'm working on them.

@Lukortech , sorry, but I can only work on this project on weekends.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants