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

feat(typography): implement typography functions and mixins to the material spec #2112

Merged
merged 43 commits into from
Aug 22, 2018

Conversation

simeonoff
Copy link
Collaborator

Closes #1351

Breaking Changes:

  • Applications using Ignite UI for Angular now require the igx-typography class to be applied on wrapping element, like the body element for instance.

desig9stein
desig9stein previously approved these changes Jul 30, 2018
wnvko
wnvko previously approved these changes Aug 1, 2018
desig9stein
desig9stein previously approved these changes Aug 9, 2018
@zdrawku zdrawku self-assigned this Aug 20, 2018
@zdrawku zdrawku added 💥 status: in-test PRs currently being tested and removed ❌ status: awaiting-test PRs awaiting manual verification labels Aug 20, 2018
Copy link
Contributor

@zdrawku zdrawku left a comment

Choose a reason for hiding this comment

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

@simeonoff, kudos for the great work that you have done with the Typography in Ignite UI for Angular!

Documentation, Readme.md and implementation of all mixins and functions are very well explained and intuitive to use.

I strongly hope that out clients will benefit of this powerful type system and will use it with ease.

I have uploaded part of my testing implementation in a GitHub repository

@zdrawku zdrawku added ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Aug 22, 2018
@zdrawku zdrawku merged commit 6bd228a into master Aug 22, 2018
@zdrawku zdrawku deleted the typography branch August 22, 2018 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔡 typography feature version: 6.2.x ✅ status: verified Applies to PRs that have passed manual verification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants