Bug 1374251 - Save current title into data model before navigating to SelectFolderFragment. r?Grisha draft
authorjwu <topwu.tw@gmail.com>
Thu, 29 Jun 2017 14:10:37 -0700
changeset 602275 e21ba07822c472f2ff32f97dee85734d1b6773dd
parent 602272 967aac2f682fae004e7575ee340c63503eac9807
child 635520 2fd073414f76bf55a91a8e3320039d16182100f8
push id66345
push userbmo:topwu.tw@gmail.com
push dateThu, 29 Jun 2017 21:11:47 +0000
reviewersGrisha
bugs1374251
milestone56.0a1
Bug 1374251 - Save current title into data model before navigating to SelectFolderFragment. r?Grisha MozReview-Commit-ID: JfPWi0HV6eB
mobile/android/base/java/org/mozilla/gecko/bookmarks/BookmarkEditFragment.java
--- a/mobile/android/base/java/org/mozilla/gecko/bookmarks/BookmarkEditFragment.java
+++ b/mobile/android/base/java/org/mozilla/gecko/bookmarks/BookmarkEditFragment.java
@@ -164,16 +164,20 @@ public class BookmarkEditFragment extend
 
         folderText.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View v) {
                 if (bookmark == null) {
                     return;
                 }
 
+                // When coming back from SelectFolderFragment, we update view with data stored in `bookmark`,
+                // so before navigating, we have to save current title from nameText into `bookmark`.
+                bookmark.title = nameText.getText().toString();
+
                 final SelectFolderFragment dialog = SelectFolderFragment.newInstance(bookmark.parentId, bookmark.id);
                 dialog.setTargetFragment(BookmarkEditFragment.this, 0);
                 dialog.show(getActivity().getSupportFragmentManager(), "select-bookmark-folder");
             }
         });
 
         return view;
     }