10

I have a layout that has an EditText and a Button. I <include> it in my main layout.

I'm having a weird issue with the layout and rotation. It seems to duplicate itself when the device (physical) is rotated, messing up the text and layout.

Here it is on first open, after I add some extra garble:

1

DSC_0013 is in the EditText on launch of the fragment.

Then, I rotate the phone and add some different garble:

2

And you can see the issue pretty clearly. At first, I thought it was just the EditText messing up. But if I add enough text to make a new line:

3

I can see that the button gets messed up too.

I do override onSaveInstanceState, but in it I don't touch the EditText or its value, it's strictly used for something else.

What's happening and how do I fix it?

1
  • How do you set the view? setContentView + XML? addView exclusively?
    – Cat
    Sep 10, 2012 at 0:08

3 Answers 3

16

Fixed it!

Turns out it wasn't the view duplicating itself, or the EditText, or the Button. It was the entire fragment.

In my Activity's onCreate, I add the fragment to an xml layout:

private FileDetails fileDetailsFragment;

public void onCreate(Bundle savedInstanceState) {
        ...
        FragmentManager fragmentManager = getSupportFragmentManager();
        FragmentTransaction fragmentTransaction = fragmentManager
                .beginTransaction();
        fileDetailsFragment = new FileDetails(fileData);
        fragmentTransaction.add(R.id.DetailsHolder, fileDetailsFragment);
        fragmentTransaction.commit();

And onCreate was being called every time I rotated the phone (as it's meant to). So I put in a check to see if the activity is being run for the first time, and it works great.

private FileDetails fileDetailsFragment;

public void onCreate(Bundle savedInstanceState) {
    ...
    if (savedInstanceState == null) {
        FragmentManager fragmentManager = getSupportFragmentManager();
        FragmentTransaction fragmentTransaction = fragmentManager
                .beginTransaction();
        fileDetailsFragment = new FileDetails(fileData);
        fragmentTransaction.add(R.id.DetailsHolder, fileDetailsFragment);
        fragmentTransaction.commit();
    } else {
        fileDetailsFragment = (FileDetails) getSupportFragmentManager().findFragmentById(R.id.DetailsHolder);
    }
3
  • 4
    Over a year later and this solution just helped me out :) Thanks very much for posting this!
    – Alex
    Nov 11, 2013 at 20:54
  • 1
    Now over three years ^^
    – Tomblarom
    Aug 7, 2015 at 11:02
  • 1
    Still very useful answer Aug 15, 2016 at 8:41
1

You can also setRetainedInstance(true) on your fragment, then try to get the Fragment form de FragmentManager.findFragmentById(int) or FragmentManager.findFragmentByTag(String), and if it returns null it meant you had to create a new instance of your Fragment.

private FileDetails fileDetailsFragment; 


public void onCreate(Bundle savedInstanceState) {
...
    FragmentManager fragmentManager = getSupportFragmentManager();
    fileDetailsFragment = (FileDetails) getSupportFragmentManager().findFragmentById(R.id.DetailsHolder);
    if (fileDetailsFragment == null) {
        fileDetailsFragment = new FileDetails(FileData); 
    }

    FragmentTransaction fragmentTransaction = fragmentManager
            .beginTransaction();

    fragmentTransaction.add(R.id.DetailsHolder, fileDetailsFragment);
    fragmentTransaction.commit();
}
0

In some cases, the value of savedInstanceState may be null after rotation, so it is better to add another condition:

FragmentManager fragmentManager = getSupportFragmentManager();
if (savedInstanceState == null && 
    fragmentManager.getFragments().size() == 0) {
        
        FragmentTransaction fragmentTransaction = fragmentManager
                .beginTransaction();
        fileDetailsFragment = new FileDetails(fileData);
        fragmentTransaction.add(R.id.DetailsHolder, fileDetailsFragment);
        fragmentTransaction.commit();
    } else {
        fileDetailsFragment = (FileDetails) 
        getSupportFragmentManager().findFragmentById(R.id.DetailsHolder);
    }

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.