Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Md-inputs not showing text typed into them #7078

Closed
jpike88 opened this issue Feb 9, 2016 · 4 comments
Closed

Md-inputs not showing text typed into them #7078

jpike88 opened this issue Feb 9, 2016 · 4 comments
Labels
has: Pull Request A PR has been created to address this issue os: iOS This issue is specific to iOS P1: urgent Urgent issues that should be addressed in the next minor or patch release. resolution: fixed ui: browser
Milestone

Comments

@jpike88
Copy link
Contributor

jpike88 commented Feb 9, 2016

This is a serious usability issue and requires top priority as there are plenty of use cases which would have a form start off the page.

This video shows the problem.

https://www.dropbox.com/s/428d9p0947kt46i/Untitled.mov?dl=0

@jpike88
Copy link
Contributor Author

jpike88 commented Feb 9, 2016

@ThomasBurleson You mentioned in another thread that changing the style to '-webkit-overflow-scrolling: auto' solves the issue, which it does at the expense of usability.

Is this a webkit bug or an md-input-container bug, and what is the next step, as that is just a workaround?

@topherfangio
Copy link
Contributor

@jpike88 As far as I can tell, it is a Webkit bug as it can affect things other than the input container; that just happens to be one of the most noticeable.

I was looking into this a while ago, and if I recall correctly, I was able to fix the issue by adding that autoscroll CSS to a parent of the inputs, but a child of the content. So, something like this seemed to work:

<md-content>
    <div style="-webkit-overflow-scrolling: auto;">
        <md-input-container><input /></md-input-container>
        <md-input-container><input /></md-input-container>
        <md-input-container><input /></md-input-container>
    </div>
</md-content>

Can you give this a test and see if it works for you?

@jpike88
Copy link
Contributor Author

jpike88 commented Feb 11, 2016

Weird it works.

I think this needs to be hardcoded in ng-material, there's no way I would have found that workaround out on my own.

@topherfangio
Copy link
Contributor

@jpike88 If we can find a simple/backwards-compatible way to include it, I totally agree that it should be added. Perhaps at the very least we may be able to add it to the input component itself since it seems to be the worst offender.

@topherfangio topherfangio added ui: browser needs: team discussion This issue requires a decision from the team before moving forward. - Known Issue P1: urgent Urgent issues that should be addressed in the next minor or patch release. labels Feb 11, 2016
@topherfangio topherfangio added this to the 1.1.0 milestone Feb 11, 2016
@topherfangio topherfangio self-assigned this Feb 11, 2016
topherfangio added a commit to profoundry-us/material that referenced this issue Jun 6, 2016
Often times when scrolling on an iOS device, certain elements on
the page would flicker. This was most noticeable with inputs, but
happens elsewhere too.

Add a new `md-no-flicker` CSS class which is automatically applied
to the `<md-content>` which adds a non-changing CSS transform to
the element which fixes the flicker.

Update docs to mention new class and workarounds if it causes
issues for developers.

Fixes angular#7078.
@topherfangio topherfangio added the has: Pull Request A PR has been created to address this issue label Jun 6, 2016
@Splaktar Splaktar added the os: iOS This issue is specific to iOS label May 9, 2019
@Splaktar Splaktar added resolution: fixed and removed needs: team discussion This issue requires a decision from the team before moving forward. labels May 9, 2019
@angular angular locked as resolved and limited conversation to collaborators May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has: Pull Request A PR has been created to address this issue os: iOS This issue is specific to iOS P1: urgent Urgent issues that should be addressed in the next minor or patch release. resolution: fixed ui: browser
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants