react / react/react-native

Android TextInput placeholder with custom font has incorrect height

Open
#31,544 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Component: TextInput Issue: Author Provided Repro
Dominant language
C++
Stars
127k
Forks
25.3k
Avg merge
1d 23h
Merged PRs (30d)
4

Description

Description

TextInput placeholders that do not use the default android font and differ in their wrap point from the default font may be cut off. For example, if the default font does not wrap, and the custom font is slightly wider and does wrap to the next line, the placeholder will NOT reflect this wrapping, and the TextInput will only be one line.

React Native version:

System:
    OS: macOS 11.2.3
    CPU: (8) x64 Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz
    Memory: 91.84 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.0.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.10.0 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
    Android SDK:
      API Levels: 28, 29, 30
      Build Tools: 28.0.3, 29.0.2, 30.0.3, 31.0.0
      System Images: android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.8743.12.41.6953283
    Xcode: 12.4/12D4e - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.10 - /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.1 => 17.0.1 
    react-native: 0.64.1 => 0.64.1 
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps To Reproduce

  • Create text input with placeholder string that does not wrap with default font, but does wrap with custom font.
  • Second line will not be visible.

Expected Results

Placeholder should adjust height according to its font.

Snack, code example, screenshot, or link to a repository:

https://snack.expo.io/@computerjazz/textinput-android-bug

expected behavior (in this case placeholder text is long enough that it ALSO would wrap using default font)
Screen Shot 2021-05-18 at 9 27 19

actual behavior (placeholder text using "serif" font wraps to a 3rd line, but if it were to use the default font it would not wrap, and as a result placeholder gets cut off):
Screen Shot 2021-05-18 at 9 22 40

For reference, here's the same text using default font, note that it does not wrap to a 3rd line:
Screen Shot 2021-05-18 at 9 30 30

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the wrapping difference with the linked Snack, using the custom serif font and a placeholder that wraps differently from the default font. Then trace the Android React Native TextInput implementation responsible for placeholder sizing and verify that the placeholder height includes all wrapped lines.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, react-native
Domain
mobile
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.