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

add declaration file (index.d.ts) #151

Merged
merged 7 commits into from
Mar 29, 2021
Merged

Conversation

long-hp
Copy link
Contributor

@long-hp long-hp commented Dec 18, 2019

Adding declaration file helps editor suggest props for you and report errors typescript

Screenshot 2019-12-19 00 21 23
Screenshot 2019-12-19 00 49 38
Screenshot 2019-12-19 10 34 11
Screenshot 2019-12-19 12 15 06
Screenshot 2019-12-19 12 15 28

@palingheorghe
Copy link
Collaborator

Thank you for this PR! This will sure come in handy for our users! I'll have to review it and then add it in Galio's next version. This will probably happen after New Year Eve.

You've done a great help to our community ❤️!

@long-hp
Copy link
Contributor Author

long-hp commented Dec 20, 2019

@palingheorghe I don't know icon font type is correct or not? But some fonts are not working
I need exactly the type of icon font family. Can you give it to me?

Some of the prop types in your document are incorrect so I can hardly write them correctly

type IconFamilyType =
    | 'Galio'
    | 'AntDesign'
    | 'Entypo'
    | 'EvilIcons'
    | 'Feather'
    | 'FontAwesome'
    | 'FontAwesome5'
    | 'Fontisto'
    | 'Foundation'
    | 'Ionicons'
    | 'MaterialIcons'
    | 'MaterialCommunityIcons'
    | 'Octicons'
    | 'Zocial'
    | 'SimpleLineIcons';

@palingheorghe
Copy link
Collaborator

Hi @wiloke1 ! You can find more about the font family type names here: https://github.com/galio-org/galio/blob/master/src/helpers/getIconType.js . These must be: zocial, octicon, material, material-community, ionicon, foundation, evilicons, entypo, font-awesome, font-awesome-5, simple-line-icon, feather, antdesign. On top of that we have Galio as one of the font family names.

@superandrew
Copy link

👍 adding myself to that, is this going to be released anytime soon?

@AndyOsei
Copy link
Collaborator

We are working on releasing hotfixes for immediate pending issues, will discuss on considering adding this pull request to that list with the others after review...

Am confident it will be added pretty soon...

@kevbradwick
Copy link

just wanted to add my support for this PR - I could really do with it right now 👍

shadow?: boolean;
borderless?: boolean;
image?: string;
imageBlockStyle?: strubg;
Copy link

Choose a reason for hiding this comment

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

what is strubg?

rounded?: boolean;
borderless?: boolean;
viewPass?: boolean;
icon?: string;
Copy link

Choose a reason for hiding this comment

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

@RommelTJ RommelTJ mentioned this pull request Aug 26, 2020
RommelTJ added a commit to RommelTJ/galio that referenced this pull request Aug 26, 2020
@SapioBeasley
Copy link

merge please .. please? lol

@palingheorghe palingheorghe changed the base branch from master to dev March 29, 2021 11:33
@palingheorghe palingheorghe merged commit 79a023f into galio-org:dev Mar 29, 2021
@palingheorghe palingheorghe mentioned this pull request Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants