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

Fixed scroll jumping up when typing past field height #315

Closed
wants to merge 19 commits into from

Conversation

Skquark
Copy link

@Skquark Skquark commented May 23, 2020

Refer to #171

masewo and others added 18 commits November 7, 2019 20:16
…ace-master

# Conflicts:
#	packages/zefyr/lib/src/widgets/input.dart
…en typing long

Described in memspace#171
Solution by raj457036
Haven't tested yet, not sure about the isAtEnd logic...
Correcting my mistakes after I got errors trying it out
ScrollController not attached to any scroll views.
'package:flutter/src/widgets/scroll_controller.dart':
Failed assertion: line 110 pos 12: '_positions.isNotEmpty'
@codecov
Copy link

codecov bot commented May 23, 2020

Codecov Report

Merging #315 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #315   +/-   ##
=======================================
  Coverage   91.89%   91.89%           
=======================================
  Files          11       11           
  Lines        1074     1074           
=======================================
  Hits          987      987           
  Misses         87       87           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36b3669...761acfe. Read the comment docs.

Got rid of showAutocorrectionPromptRect & AutofillScope, which I'm not sure what it was serving when unimplemented.
Was from masewo's changes.
Copy link
Author

@Skquark Skquark left a comment

Choose a reason for hiding this comment

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

Deleted the parts that caused the build failure. I should of forked from the official repo instead of masewo's I was using...

Skquark added a commit to Skquark/zefyr that referenced this pull request May 24, 2020
To replace my original PR memspace#315
I forked from the wrong fork, and wanted to simplify commits to one, and fix the Travis CL build failure it was reporting.
Issue discussed here memspace#171
@Skquark
Copy link
Author

Skquark commented May 24, 2020

Simplified the fork and resubmitted the Pull.. Sorry.

@Skquark Skquark closed this May 24, 2020
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