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

Editor crashes when my app was updated to latest xamarin.forms 3.4 .0.1008975 #4832

Closed
YongqingKyleHuang opened this issue Dec 20, 2018 · 9 comments · Fixed by #5653
Closed
Labels
i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often i/regression p/Android t/bug 🐛
Projects

Comments

@YongqingKyleHuang
Copy link

Description

My App crashes by using Editor, when I updated xamarin.forms from 3.2 to3.4 .

It crashes only when my smartphone's orientation is landscape.

It happens only by Andriod. There is no such a bug by iOS.

Steps to Reproduce

  1. Andriod, uses portable project, one of tagets is .NET Framework 4.5
  2. smartphone's orientation is landscape ( portrait is ok)
  3. Input some text for Editor then press "DONE"
  4. Crashes:

Java.Lang.IllegalStateException: focus search returned a view that wasn't able to take focus!

Expected Behavior

Basic Information

  • Version with issue:

  • Last known good version:

  • IDE:
    Visual Studio 2017

  • Platform Target Frameworks:

    • Android: < 7.1 -->
@YongqingKyleHuang
Copy link
Author

Affected device: Moto G(5S)

@kingces95 kingces95 added the s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. label Dec 20, 2018
@kingces95
Copy link
Contributor

Please attached a zipped reproduction. Ideally reduced (start with a blank template project).

@kingces95 kingces95 moved this from New to Needs Info in Triage Dec 20, 2018
@samhouts
Copy link
Member

@kylehuang818a Since we haven't heard from you in more than 30 days, we hope this issue is no longer affecting you. If it is, please reopen this issue and provide the requested information so that we can look into it further. Thank you!

Triage automation moved this from Needs Info to Closed Jan 22, 2019
@tdduy
Copy link

tdduy commented Jan 24, 2019

I confirm that issue still exists in Xamarin.Forms 3.4 .0.1009999.

@byishimoaudace
Copy link

byishimoaudace commented Feb 5, 2019

The issue still exists in Xamarin.Forms 3.4.0.1029999 on SONY Xperia devices

@Brosten
Copy link

Brosten commented Feb 18, 2019

Very easy to reproduce. New Xamarin Forms Project,
XF version 3.5
Setup page like this:

    <StackLayout>
      <Editor/>
      <Button Text="SomeText" IsEnabled="False"/>
   </StackLayout>

Run on Android phone.
Put it in landscape
Enter edit mode of the editor
Push "Done" button.
Crash

Tested with Samsung Galaxy S7 running Android 8.1. Google keyboard.
@samhouts

@samhouts samhouts reopened this Feb 19, 2019
Triage automation moved this from Closed to New Feb 19, 2019
@samhouts samhouts added p/Android i/regression i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often and removed s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. labels Feb 19, 2019
@pauldipietro
Copy link
Contributor

Those steps cause the aforementioned crash on an API 27 emulator.

@pauldipietro pauldipietro moved this from New to Ready For Work in Triage Feb 20, 2019
@Brosten
Copy link

Brosten commented Feb 20, 2019

A lazy workaround I'm using until fixed in XF

   <StackLayout>
      <Editor/>
      <Editor x:Name="CrashStopperHack" 
                    HeightRequest="0" 
                    WidthRequest="0"
                    Focused="CrashStopperHack_Focused"/>
      <Button Text="SomeText" IsEnabled="False"/>
   </StackLayout>

And in code behind

      private void CrashStopperHack_Focused(object sender, FocusEventArgs e)
      {
         CrashStopperHack.Unfocus();
      }

@andreinitescu
Copy link
Contributor

andreinitescu commented Feb 20, 2019

Should be fixed by #5320

@samhouts samhouts added this to In Progress in v4.0.0 Feb 20, 2019
@samhouts samhouts removed this from Ready For Work in Triage Feb 21, 2019
@samhouts samhouts added this to In Progress in v4.1.0 Mar 2, 2019
@samhouts samhouts removed this from In Progress in v4.0.0 Mar 2, 2019
StephaneDelcroix pushed a commit that referenced this issue Mar 26, 2019
* Set ImeOptions to Done on Android Editor

* Apply fix to Material Editor renderer

- fixes #4832
- fixes #5030
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Mar 27, 2019
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Mar 27, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Mar 27, 2019
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Mar 28, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Mar 29, 2019
adrianknight89 pushed a commit to adrianknight89/Xamarin.Forms that referenced this issue Mar 29, 2019
* Set ImeOptions to Done on Android Editor

* Apply fix to Material Editor renderer

- fixes xamarin#4832
- fixes xamarin#5030
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Apr 1, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Apr 2, 2019
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Apr 3, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Apr 3, 2019
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Apr 3, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Apr 4, 2019
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Apr 4, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Apr 4, 2019
@samhouts samhouts moved this from Done to In Progress in v4.1.0 Apr 5, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Apr 5, 2019
codemillmatt pushed a commit to codemillmatt/Xamarin.Forms that referenced this issue Apr 19, 2019
* Set ImeOptions to Done on Android Editor

* Apply fix to Material Editor renderer

- fixes xamarin#4832
- fixes xamarin#5030
@samhouts samhouts added this to In Progress in v4.2.0 May 29, 2019
@samhouts samhouts removed this from Done in v4.1.0 May 29, 2019
@samhouts samhouts moved this from In Progress to Done in v4.2.0 May 29, 2019
@samhouts samhouts removed this from Done in v4.2.0 Jun 4, 2019
@samhouts samhouts added this to In Progress in v4.1.0 Jun 4, 2019
@samhouts samhouts moved this from In Progress to Done in v4.1.0 Jun 5, 2019
AxelUser pushed a commit to AxelUser/Xamarin.Forms that referenced this issue Jun 15, 2019
* Set ImeOptions to Done on Android Editor

* Apply fix to Material Editor renderer

- fixes xamarin#4832
- fixes xamarin#5030
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often i/regression p/Android t/bug 🐛
Projects
No open projects
v4.1.0
  
Done
Development

Successfully merging a pull request may close this issue.

8 participants