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

Floating label alpha evaluation #275

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Floating label alpha evaluation #275

wants to merge 3 commits into from

Conversation

vadymhimself
Copy link

Fixed wrong alpha calculation for floating label.

    <com.rengwuxian.materialedittext.MaterialEditText
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        app:met_floatingLabel="normal"
        app:met_textColorHint="#000"
        app:met_floatingLabelTextColor="#000"
        android:hint="Hello World!"/>

Expected result should be:
image

Actual result:
image

Fixed by correcting floating label alpha evaluation

@Rainer-Lang
Copy link

Thanks - I hope this will be fast included.

@vadymhimself
Copy link
Author

@rengwuxian please include this. Right now it is impossible to have expected color on floating label.

@iam1492
Copy link

iam1492 commented Mar 30, 2016

+1

1 similar comment
@jimsmithm3
Copy link

+1

AaronRietschlin pushed a commit to AbercrombieAndFitch/MaterialEditText that referenced this pull request May 25, 2016
@vanillazz
Copy link

+1

@gchristov
Copy link

When will this be merged? It is very needed because the library doesn't look good on dark backgrounds..

Copy link

@choongyouqi choongyouqi left a comment

Choose a reason for hiding this comment

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

good

@bass3l
Copy link

bass3l commented Dec 29, 2016

+1

3 similar comments
@felipecastilhos
Copy link

+1

@lurecas
Copy link

lurecas commented Aug 11, 2017

+1

@fitnessthetic
Copy link

+1

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.