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: add theme extensions #203

Merged
merged 16 commits into from
Dec 29, 2023
Merged

feat: add theme extensions #203

merged 16 commits into from
Dec 29, 2023

Conversation

SolMendiola
Copy link
Contributor

⭐ Feature


✏️ Description:

Add theme extensions of colors, typography and dimensions.


@SolMendiola SolMendiola mentioned this pull request Dec 19, 2023
8 tasks
catalog/lib/theme/custom_colors.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_theme.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_theme.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_theme.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_dimensions.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/common/helper.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/theme/custom_colors.dart Outdated Show resolved Hide resolved
catalog/lib/theme/custom_colors.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_theme.dart Outdated Show resolved Hide resolved
catalog/lib/extensions/color_extensions.dart Outdated Show resolved Hide resolved
catalog/lib/extensions/color_extensions.dart Outdated Show resolved Hide resolved
catalog/lib/extensions/color_extensions.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
@SolMendiola SolMendiola removed the request for review from f7deleon December 28, 2023 14:30
catalog/lib/common/helper.dart Show resolved Hide resolved
catalog/lib/common/helper.dart Show resolved Hide resolved
catalog/lib/theme/app_buttons.dart Outdated Show resolved Hide resolved
catalog/lib/extensions/color_extensions.dart Show resolved Hide resolved
: _generateMaterialColor();

MaterialColor _generateMaterialColor() => MaterialColor(value, {
50: _tintColor(0.9),
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be shade instead of tint? whats the difference?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The difference it is the way of calculate tint and shade

catalog/lib/extensions/color_extensions.dart Outdated Show resolved Hide resolved
catalog/lib/theme/app_dimensions.dart Outdated Show resolved Hide resolved
final Color? warning;
final Color? danger;

static CustomColors getCustomColors() => const CustomColors(
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this necessary, isn't it accessible by Theme.of(context).extension<MyColors>()!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would we like to access the colors that way?

@SolMendiola SolMendiola merged commit 539be40 into main Dec 29, 2023
2 checks passed
@SolMendiola SolMendiola deleted the feature/add-theme-extensions branch December 29, 2023 14:16
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.

3 participants