faciocode / design Goto Github PK

Open Source Design System by Facio

License: ISC License

Dart 18.64% JavaScript 8.77% TypeScript 70.48% Shell 0.13% HTML 1.55% Kotlin 0.03% Swift 0.09% Objective-C 0.01% Ruby 0.30%

design design-system facio open opensource ui ui-components

design's People

Contributors

Stargazers

Watchers

Forkers

dartganan murilosandiego baronrustamov

design's Issues

PR checks should verify if all required generated files are present

Today, we can open and merge a PR without all the required generated files that are required for lib publication.

example:
the PR #432 was merged, the lib built and it failed only on pub.dev publication due to an error on the example project, that was using the new version.

When a PR is open there should be a check to verify if the required files.

bug(flutter-styles): renome TextStyles to another name

@jimmyandrade the name TextStyles is reserved to an official Flutter library so it must be changed.

design/packages/flutter-styles/lib/text_theme.dart

Line 13 in 3f9a5b9

class TextStyles {

Check if we will use double or single for label margin top",

design/packages/style-dictionary/src/tokens/globals/component/filled_input.json

Lines 32 to 37 in 2f08808

 "comment": "TODO: Check if we will use double or single for label margin top", 

 "value": "{size.base.single.value}" 

 } 

 }, 

 "input": { 

 "paddingBottom": { "value": "{size.base.singleHalf.value}" },

This issue was generated by todo based on a `TODO` comment in `2f08808`. It's been assigned to @jimmyandrade because they committed the code.

fix(styles): check values for font size, letter spacing and line height

design/packages/styles/src/tokens/platforms/web/size/font.json

Lines 108 to 113 in c0d30ed

 "comment": "Todo: check these values", 

 "fontSize": { "value": 0.875 }, 

 "letterSpacing": { "value": -0.15 }, 

 "lineHeight": { "value": "{size.font.base.lineHeight.value}" } 

 }, 

 "caption": {

This issue was generated by todo based on a `Todo` comment in `c0d30ed`. It's been assigned to @jimmyandrade because they committed the code.

adopt solid background colors in favor of opacity

design/packages/flutter_ds/lib/src/components/buttons/small_contained_button.dart

Lines 42 to 47 in 9481bc8

 // TODO: adopt solid background colors in favor of opacity 

 opacity: _isEnabled ? ButtonStyles.opacity.toDouble() : 0.5, 

 child: ConstrainedBox( 

 constraints: BoxConstraints.tightFor(height: ButtonStyles.sizeSmallHeight), 

 child: TextButton( 

 key: _key,

This issue was generated by todo based on a `TODO` comment in `9481bc8`. It's been assigned to @pandoracosta because they committed the code.

feat(react): no default jsDelivr CDN file set

This package doesn't have a default file set. You can set it via jsdelivr, browser, or main field in package.json

docs(styles): define and document use for headline2

design/packages/styles/src/tokens/globals/component/typography.json

Lines 19 to 24 in c0d30ed

 "comment": "Formerly title1. Todo: define its use.", 

 "color": { "value": "{color.font.secondary.value}" }, 

 "fontFamily": { "value": "{font.family.base.value}" }, 

 "fontSize": { 

 "comment": "Very, very large text", 

 "value": "{size.font.headline2.fontSize.value}"

This issue was generated by todo based on a `Todo` comment in `c0d30ed`. It's been assigned to @jimmyandrade because they committed the code.

adopt solid background colors in favor of opacity

design/packages/flutter_ds/lib/src/components/buttons/small_contained_button.dart

Lines 42 to 47 in c693e8e

 // TODO: adopt solid background colors in favor of opacity 

 opacity: _isEnabled ? ButtonStyles.opacity.toDouble() : 0.5, 

 child: ConstrainedBox( 

 constraints: BoxConstraints.tightFor(height: ButtonStyles.sizeSmallHeight), 

 child: TextButton( 

 key: _key,

This issue was generated by todo based on a `TODO` comment in `c693e8e` when #367 was merged. cc @FacioCode.

check if we will use grey.50 or grey.60

design/packages/style-dictionary/src/tokens/globals/component/contained_button.json

Lines 18 to 23 in fb38487

 "comment": "Todo: check if we will use grey.50 or grey.60", 

 "value": "{color.font.disabled.value}" 

 } 

 }, 

 "brandHover": { 

 "backgroundColor": { "value": "{color.base.primary.main.value}" },

This issue was generated by todo based on a `Todo` comment in `fb38487`. It's been assigned to @jimmyandrade because they committed the code.

write tests\" && exit 0"

design/packages/react/package.json

Lines 70 to 74 in 770c74d

 "test": "echo \"Todo: write tests\" && exit 0" 

 }, 

 "publishConfig": { 

 "access": "public" 

 }

This issue was generated by todo based on a `Todo` comment in `770c74d`. It's been assigned to @undefined because they committed the code.

re-enable after fixing null safety package error

design/.github/workflows/publish.yml

Lines 98 to 101 in ff7c21c

 if: ${{ false }} # TODO: re-enable after fixing null safety package error 

 needs: build 

 runs-on: ubuntu-latest 

 env:

This issue was generated by todo based on a `TODO` comment in `ff7c21c`. It's been assigned to @jimmyandrade because they committed the code.

feat(react-unstyled): no default jsDelivr CDN file set

This package doesn't have a default file set. You can set it via jsdelivr, browser, or main field in package.json

	"comment": "TODO: Check if we will use double or single for label margin top",
	"value": "{size.base.single.value}"
	}
	},
	"input": {
	"paddingBottom": { "value": "{size.base.singleHalf.value}" },

	"comment": "Todo: check these values",
	"fontSize": { "value": 0.875 },
	"letterSpacing": { "value": -0.15 },
	"lineHeight": { "value": "{size.font.base.lineHeight.value}" }
	},
	"caption": {

	// TODO: adopt solid background colors in favor of opacity
	opacity: _isEnabled ? ButtonStyles.opacity.toDouble() : 0.5,
	child: ConstrainedBox(
	constraints: BoxConstraints.tightFor(height: ButtonStyles.sizeSmallHeight),
	child: TextButton(
	key: _key,

	"comment": "Formerly title1. Todo: define its use.",
	"color": { "value": "{color.font.secondary.value}" },
	"fontFamily": { "value": "{font.family.base.value}" },
	"fontSize": {
	"comment": "Very, very large text",
	"value": "{size.font.headline2.fontSize.value}"

	"comment": "Todo: check if we will use grey.50 or grey.60",
	"value": "{color.font.disabled.value}"
	}
	},
	"brandHover": {
	"backgroundColor": { "value": "{color.base.primary.main.value}" },

	"test": "echo \"Todo: write tests\" && exit 0"
	},
	"publishConfig": {
	"access": "public"
	}

	if: ${{ false }} # TODO: re-enable after fixing null safety package error
	needs: build
	runs-on: ubuntu-latest
	env:

faciocode / design Goto Github PK

design's People

Contributors

Stargazers

Watchers

Forkers

design's Issues

This issue was generated by todo based on a TODO comment in 2f08808. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a Todo comment in c0d30ed. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a TODO comment in 9481bc8. It's been assigned to @pandoracosta because they committed the code.

This issue was generated by todo based on a Todo comment in c0d30ed. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a TODO comment in c693e8e when #367 was merged. cc @FacioCode.

This issue was generated by todo based on a Todo comment in fb38487. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a Todo comment in 770c74d. It's been assigned to @undefined because they committed the code.

This issue was generated by todo based on a TODO comment in ff7c21c. It's been assigned to @jimmyandrade because they committed the code.

Recommend Projects

Recommend Topics

Recommend Org

Jobs

This issue was generated by todo based on a `TODO` comment in `2f08808`. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a `Todo` comment in `c0d30ed`. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a `TODO` comment in `9481bc8`. It's been assigned to @pandoracosta because they committed the code.

This issue was generated by todo based on a `Todo` comment in `c0d30ed`. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a `TODO` comment in `c693e8e` when #367 was merged. cc @FacioCode.

This issue was generated by todo based on a `Todo` comment in `fb38487`. It's been assigned to @jimmyandrade because they committed the code.

This issue was generated by todo based on a `Todo` comment in `770c74d`. It's been assigned to @undefined because they committed the code.

This issue was generated by todo based on a `TODO` comment in `ff7c21c`. It's been assigned to @jimmyandrade because they committed the code.