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

Remove unwanted shadow and Add TextStyle for Tab label #12

Merged
merged 6 commits into from
Aug 19, 2019

Conversation

mderis
Copy link
Contributor

@mderis mderis commented Aug 15, 2019

  1. According to the artwork in Uplabs the icons must have no shadow and follow the flat colors.
    So I removed it and you can see the changes here:
    Screenshot_2019-08-15-06-00-42

  2. I made changes so users can pass style for label of each TabItem, of course it is not required and there is some default style for when the user leaves it empty (Bold) . This feature is added to fix issue Textsize of Menu titile #6

@mderis mderis mentioned this pull request Aug 15, 2019
@imaNNeo
Copy link
Collaborator

imaNNeo commented Aug 15, 2019

Hi dude,
may I ask you to make the necessary changes on README.md?

@mderis
Copy link
Contributor Author

mderis commented Aug 15, 2019

@imaNNeoFighT Sure! What needs to be changed there?

@imaNNeo
Copy link
Collaborator

imaNNeo commented Aug 15, 2019

We need to update sample codes, and also if you have sketch in your machine, please update the CheatSheet (using this file).

@mderis
Copy link
Contributor Author

mderis commented Aug 15, 2019

I have updated the sample code in this PR already but I'm afraid there is no possibility for me to modify the file you just mentioned.

@mderis
Copy link
Contributor Author

mderis commented Aug 16, 2019

Aint it gonna be merged? :-?

@imaNNeo
Copy link
Collaborator

imaNNeo commented Aug 17, 2019

Yeap, just wait

@imaNNeo imaNNeo merged commit 7a8b127 into benyaminbeyzaie:master Aug 19, 2019
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.

2 participants