-
Notifications
You must be signed in to change notification settings - Fork 13
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
Figma generator #1
Conversation
packages/theme_tailor_figma_loader/lib/src/models/app_color.dart
Outdated
Show resolved
Hide resolved
packages/theme_tailor_figma_loader/lib/src/models/app_color.dart
Outdated
Show resolved
Hide resolved
packages/theme_tailor_figma_loader/lib/src/models/app_typography.dart
Outdated
Show resolved
Hide resolved
packages/theme_tailor_figma_loader/lib/src/models/app_color.dart
Outdated
Show resolved
Hide resolved
packages/theme_tailor_figma_loader/lib/src/templates/class_color_template.dart
Outdated
Show resolved
Hide resolved
packages/theme_tailor_figma_loader/lib/src/templates/class_typography_template.dart
Outdated
Show resolved
Hide resolved
final s = ((lineHeight ?? 1) / (fontSize ?? 1)).toStringAsFixed(2); | ||
final returnValue = double.parse(s); | ||
return returnValue; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would not round it. here and just use toStringAsFixed(2) if needed when it is converted to string
I will add tommorow example and pictures to the readme |
@@ -0,0 +1 @@ | |||
2.10.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it shouldn't be version controlled, as well as other files form .dart_tools
# Generated by pub on 2022-05-30 12:26:33.147476. | ||
args:file:///Users/jlukas/.pub-cache/hosted/pub.dartlang.org/args-2.3.1/lib/ | ||
async:file:///Users/jlukas/.pub-cache/hosted/pub.dartlang.org/async-2.8.2/lib/ | ||
boolean_selector:file:///Users/jlukas/.pub-cache/hosted/pub.dartlang.org/boolean_selector-2.1.0/lib/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this file also shouldn't version controlled
7845c1e
to
4b6599d
Compare
4b6599d
to
7e86083
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are few linter warnings to be fixed, these are not crucial tho and can be fixed later (e.g. sorting constructors first (and factories) )
There are warnings about prints in the code. I think we can replace 'print' with 'log' from dart:developer
packages/theme_tailor/pubspec.yaml
Outdated
flutter: | ||
sdk: flutter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be removed and it's not used
No description provided.