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

[TextField] Form autofill doesn't update floating label text on textfield #718

Closed
illogikal opened this issue May 29, 2015 · 75 comments
Closed
Labels
bug 🐛 Something doesn't work component: text field This is the name of the generic UI component, not the React module! external dependency Blocked by external dependency, we can’t do anything about it v0.x

Comments

@illogikal
Copy link

When you use chrome to autofill text fields, the floating label text doesn't animate and show at all (as of 0.8.0)

@illogikal
Copy link
Author

This is probably a good place to start...

@hai-cea hai-cea changed the title Form autofill doesn't update floating label text on textfield [TextField] Form autofill doesn't update floating label text on textfield Jun 19, 2015
@shaurya947
Copy link
Contributor

@illogikal could you post of gif that?

@alitaheri alitaheri added the bug 🐛 Something doesn't work label Dec 8, 2015
@mbrookes
Copy link
Member

Here you go:

image

The yellow highlight indicates the fields have been autofilled. As you can see, the password is filled, but the label is still in place.

@newoga
Copy link
Contributor

newoga commented Dec 28, 2015

Mm, that's interesting. @mbrookes Do you know if it's for all input types or just password inputs?

@mbrookes
Copy link
Member

Not sure - I tried swapping the order of address and password, but autofill didn't fire, either to fill in a known user/password, or to save a new one. I'll have to figure out how best to test other fields.

@newoga
Copy link
Contributor

newoga commented Dec 28, 2015

Okay, I plan on revisiting refactoring some of the TextField stuff in the near future. I'll try to experiment a bit too. Let me know if you notice anything else.

@simensen
Copy link

Hello! We're running into this issue. Is there anything we can do locally to try and experiment w/ some potential fixes for this? Or is it going to have to wait on a refactor?

@simensen
Copy link

We have the exact problem with the password field listed above. In addition to that we have a similar problem with a plain TextField. We have hint text set which goes away if you type values in. However, if we set a value when the page renders (deep link) the hint text is still visible behind the actual value. Is this likely the same issue or would that be something separate?

image

@oliviertassinari
Copy link
Member

Is this likely the same issue or would that be something separate?

That looks like a different issue.

@antoinerousseau
Copy link
Contributor

@antoinerousseau
Copy link
Contributor

that code could be added to onChange to update a state.isAutofilled

@liesislukas
Copy link
Contributor

Chrome Version 49.0.2623.87 (64-bit) OS X El Capitan

image

if any key is pressed or mouse clicked - it floats correctly, but initial load is broken.

@buunguyen
Copy link

Exact same problem:

s

@cezarsmpio
Copy link

Any idea, guys?

@wojtkowiak
Copy link

Experiencing the same issue.

@kand617
Copy link

kand617 commented May 11, 2016

Is there a way to disable autofill that works with MUI?

@mbrookes
Copy link
Member

@kand617

Simply create a couple of fields and make them hidden with "display:none". Example:

Then put your real fields underneath.

http://stackoverflow.com/questions/15738259/disabling-chrome-autofill

@devdlabs
Copy link

any solution to fire onChange event ?

@antoinerousseau
Copy link
Contributor

@devdlabs https://github.com/callemall/material-ui/pull/3372/files

@devdlabs
Copy link

@antoinerousseau Is this going to be merged in master?

@mbrookes
Copy link
Member

@devdlabs #3372 (comment)

@cacieprins
Copy link

This might be a similar issue (same root cause, different manifestation?)

I'm not seeing the animation issues as mentioned, but the height of the TextField container does not take into account the top margin on the input. This results in the TextField extending 14px below the container. It's a fairly straightforward workaround, but I'm only seeing it with autocomplete:

screen shot 2016-06-22 at 9 58 04 am
screen shot 2016-06-22 at 9 58 16 am
screen shot 2016-06-22 at 9 58 27 am
screen shot 2016-06-22 at 9 58 34 am

@nathanmarks
Copy link
Member

This will be fixed soon ;)

image

@antoinerousseau
Copy link
Contributor

nice @nathanmarks, thanks!!

and did you find a hack to update the floating label when chrome autofills without clicking the page, too?

@adamtal3
Copy link

This is still a pretty painful issue.

I tried using vanilla-autofill-event which just didn't work.

I'm using redux-form (like many others) and I came around with an ugly workaround (just for my login form, I don't care about other forms).
I tested it only in chrome so use it under consideration.

When I added an hidden username and password fields chrome ignored the entire form (unless hiding them with display:none which chrome didn't care about).

So I used 4 ( 😞 ) extra fields. 2 to make chrome ignore my form (autocomplete=off didn't work) and 2 more in a different fake form to make chrome fill, than in componentDidMount I added a timeout that copied the values from the fake fields to the real ones using redux-form change event:

class Login extends Component {
  // This was tested under chrome only. It's ugly, but it works. Code was modified a bit for simplicity so it might not work out of the box.

  componentDidMount() {
    // Fix chrome auto-fill
    setTimeout(() => {
      const { change, dispatch }= this.props;
      if (this.refs.usernameAutoFill.value && ! this.refs.username.value) {
        dispatch(change('username', this.refs.usernameAutoFill.value));
      }
      if (this.refs.passwordAutoFill.value && !this.refs.password.value) {
        dispatch(change('password', this.refs.passwordAutoFill.value));
      }
    }, 500);
  }

  render() {
    const styles = {
      autofill: {
        height: 0,
        width: '1px',
        position: 'absolute',
        left: 0,
        top: 0
      }
    };

    return (
      <div>
        <form style={styles.autofill}>
          <input type="text" ref="usernameAutoFill" name="usernameAutoFill" />
          <input type="password" ref="passwordAutoFill" name="passwordAutoFill" />
        </form>
        <form autocomplete="off">
          <div style={styles.autofill}><input type="text" name="fakeusername" /><input type="password" name="fakepassword" /></div>
          <Field name="username" component={() => <TextField ref="username" name="username" floatingLabelText="Username" />}/>
          <Field name="password" component={() => <TextField ref="password" name="password" type="password" floatingLabelText="Password" />}/>
          <RaisedButton primary={true} label="Login" type="submit"/>
        </form>
      </div>
    )
  }
}
export default {
  Form: reduxForm({
    form: 'Login'
  })(Login)
}

I've modified the code for simplicity so use it just for reference.

@mbrookes
Copy link
Member

@tsmirnov Time for you to get stuck in and fix it then. 😜

@ozturkcangokkaya
Copy link

There is a css solution to this bug

// Selects label that comes right after the autofilled input
input:-webkit-autofill + label {
    // Insert your active label styles
}

@kishan3
Copy link

kishan3 commented Oct 8, 2018

Here you go:

image

The yellow highlight indicates the fields have been autofilled. As you can see, the password is filled, but the label is still in place.

I am facing the same issue.

@kishan3
Copy link

kishan3 commented Oct 8, 2018

There is a css solution to this bug

// Selects label that comes right after the autofilled input
input:-webkit-autofill + label {
    // Insert your active label styles
}

Can you please explain what this means ?

@ozturkcangokkaya
Copy link

There is a css solution to this bug

// Selects label that comes right after the autofilled input
input:-webkit-autofill + label {
    // Insert your active label styles
}

Can you please explain what this means ?

:-webkit-autofill selects autofilled fields by browser. Place your label right next to input and select it with :-webkit-autofill selector. If you need more help please share your current html structure.

@kishan3
Copy link

kishan3 commented Oct 9, 2018

There is a css solution to this bug

// Selects label that comes right after the autofilled input
input:-webkit-autofill + label {
    // Insert your active label styles
}

Can you please explain what this means ?

:-webkit-autofill selects autofilled fields by browser. Place your label right next to input and select it with :-webkit-autofill selector. If you need more help please share your current html structure.

Sure! Here is my HTML Please help. Thanks
https://pastebin.com/yjJCip3r

@ozturkcangokkaya
Copy link

There is a css solution to this bug

// Selects label that comes right after the autofilled input
input:-webkit-autofill + label {
    // Insert your active label styles
}

Can you please explain what this means ?

:-webkit-autofill selects autofilled fields by browser. Place your label right next to input and select it with :-webkit-autofill selector. If you need more help please share your current html structure.

Sure! Here is my HTML Please help. Thanks
https://pastebin.com/yjJCip3r

Your labels come right after the inputs, so you dont have to do anything with html.
Just select your labels right next to autofilled inputs and set them your active label styles as shown below.

.mdl-textfield__input:-webkit-autofill + .mdl-textfield__label {
    // Active label styles here
    top: -20px; // Just an example
    transform: scale(0.75); // Just an example
}

@mbrookes
Copy link
Member

mbrookes commented Oct 9, 2018

Uh, @kishan3 I think you're lost. MDL is this way --> google/material-design-lite#4827 😆

(Although given that MDL is dead Jim, perhaps you've come to the right place after all!)

@kishan3
Copy link

kishan3 commented Oct 11, 2018

Ah damn! Its confusing with names. 😆 @mbrookes One question though: can we use material UI directly into HTMLs with CDN links? I never realised MDL is dead 😢

@mbrookes
Copy link
Member

@kishan3 You can, although as it says in the docs, this means the client has to download the entire library, at least the first time until it's cached, whereas building it locally means you can optimize which components are included.

@abhisheksoni27
Copy link

What's the solution?

@mui mui locked as resolved and limited conversation to collaborators Oct 25, 2018
@oliviertassinari
Copy link
Member

oliviertassinari commented Oct 25, 2018

This issue concerns v0.x.

@mui mui unlocked this conversation Oct 31, 2018
@efstathiosntonas

This comment has been minimized.

@tom-con
Copy link

tom-con commented Dec 6, 2018

If anyone is still here, I just set this prop InputLabelProps={{ shrink: true }} on the TextField that this effected. Was the best solution for our team.

@arshmakker
Copy link

arshmakker commented Jan 10, 2019

@tom-con thanks man, it worked for me perfectly

@ymoon715
Copy link

ymoon715 commented Jan 25, 2019

screen shot 2019-01-24 at 10 47 44 pm
this problem still exists in 2019...

@rehan-sattar
Copy link

If anyone is still here, I just set this prop InputLabelProps={{ shrink: true }} on the TextField that this effected. Was the best solution for our team.
I was really confused and you solved the problem! Best recommended solution!

@oliviertassinari
Copy link
Member

This is a v0.x issue (2015), please refer to #14427.

@prakashtsi
Copy link

prakashtsi commented Apr 2, 2019

There is a css solution to this bug

// Selects label that comes right after the autofilled input
input:-webkit-autofill + label {
    // Insert your active label styles
}

@ozturkcangokkaya Hi this css solution works fine in Chrome. But when i check my form in firefox, floating label is not working. Do you have any idea why webkit-autofill affects firefox?

@oliviertassinari
Copy link
Member

Please see #14427 for v4.

@jlomoglio

This comment has been minimized.

@scatteredlife
Copy link

If anyone is still here, I just set this prop InputLabelProps={{ shrink: true }} on the TextField that this effected. Was the best solution for our team.

Thank you, I am using this: InputLabelProps={{ shrink: form.password }} , and it works perfectly.

<TextField
                id="loginPassword"
                label="Login Password"
                type="text"
                fullWidth
                value={form.password}
                onChange={(e) => setAttribute('form.password', e.target.value)}
                InputLabelProps={{ shrink: form.password }}
              />

@matsmello
Copy link

I solved my problem installing lodash.merge on my project then I realised I was filling the text field component incorrectly it was because I thought the redux behaves should be the same when I use nested objects.

As you can see, I am not filling correctly when I mixing two nested objects.

image

I solved this using the library and so it was.
image

@oliviertassinari
Copy link
Member

oliviertassinari commented Jan 28, 2020

Note that the latest version of Material-UI (v4.9.0) supports it by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: text field This is the name of the generic UI component, not the React module! external dependency Blocked by external dependency, we can’t do anything about it v0.x
Projects
None yet
Development

No branches or pull requests