Adds edit skill fragment to edit individual skills.

Android/app/src/main/java/com/majinnaibu/monstercards/ui/editmonster/EditSkillFragment.java

@@ -9,6 +9,7 @@ import android.widget.EditText;
 import androidx.activity.OnBackPressedCallback;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
+import androidx.fragment.app.Fragment;
 import androidx.lifecycle.ViewModelProvider;
 import androidx.navigation.NavBackStackEntry;
 import androidx.navigation.NavController;
@@ -19,25 +20,24 @@ import com.majinnaibu.monstercards.models.Skill;
 import com.majinnaibu.monstercards.ui.components.AbilityScorePicker;
 import com.majinnaibu.monstercards.ui.components.AdvantagePicker;
 import com.majinnaibu.monstercards.ui.components.ProficiencyPicker;
-import com.majinnaibu.monstercards.ui.shared.MCFragment;
 import com.majinnaibu.monstercards.utils.Logger;
 import com.majinnaibu.monstercards.utils.TextChangedListener;
 
-public class EditSkillFragment extends MCFragment {
+public class EditSkillFragment extends Fragment {
     private EditMonsterViewModel mEditMonsterViewModel;
     private EditSkillViewModel mViewModel;
     private ViewHolder mHolder;
     private Skill mOldSkill;
 
     @Override
-    public void onCreate(@Nullable Bundle savedInstanceState) {
+    public void onCreate(@Nullable @org.jetbrains.annotations.Nullable Bundle savedInstanceState) {
         mViewModel = new ViewModelProvider(this).get(EditSkillViewModel.class);
         if (getArguments() != null) {
             EditSkillFragmentArgs args = EditSkillFragmentArgs.fromBundle(getArguments());
+            mViewModel.copyFromSkill(args.getName(), args.getAbilityScore(), args.getProficiency(), args.getAdvantage());
             mOldSkill = new Skill(args.getName(), args.getAbilityScore(), args.getAdvantage(), args.getProficiency());
-            mViewModel.copyFromSkill(mOldSkill);
         } else {
-            Logger.logWTF("EditSkillFragment needs arguments.");
+            Logger.logWTF("This should never happen. EditSkillFragment needs arguments.");
             mOldSkill = null;
         }
         super.onCreate(savedInstanceState);
@@ -49,7 +49,10 @@ public class EditSkillFragment extends MCFragment {
         NavController navController = Navigation.findNavController(requireActivity(), R.id.nav_host_fragment);
         NavBackStackEntry backStackEntry = navController.getBackStackEntry(R.id.edit_monster_navigation);
         mEditMonsterViewModel = new ViewModelProvider(backStackEntry).get(EditMonsterViewModel.class);
+
+        // Inflate the layout for this fragment
         View root = inflater.inflate(R.layout.fragment_edit_skill, container, false);
+
         mHolder = new ViewHolder(root);
 
         mHolder.abilityScore.setValue(mViewModel.getAbilityScore().getValue());
@@ -77,19 +80,13 @@ public class EditSkillFragment extends MCFragment {
         return root;
     }
 
-    @Override
-    public void onStart() {
-        super.onStart();
-        mHolder.name.requestFocus();
-    }
-
     private static class ViewHolder {
         AbilityScorePicker abilityScore;
         AdvantagePicker advantage;
         ProficiencyPicker proficiency;
         EditText name;
 
-        ViewHolder(@NonNull View root) {
+        ViewHolder(View root) {
             abilityScore = root.findViewById(R.id.abilityScore);
             advantage = root.findViewById(R.id.advantage);
             proficiency = root.findViewById(R.id.proficiency);

Android/app/src/main/java/com/majinnaibu/monstercards/ui/editmonster/EditSkillViewModel.java

@@ -1,37 +1,57 @@
 package com.majinnaibu.monstercards.ui.editmonster;
 
-import androidx.annotation.NonNull;
 import androidx.lifecycle.LiveData;
+import androidx.lifecycle.MutableLiveData;
+import androidx.lifecycle.ViewModel;
 
 import com.majinnaibu.monstercards.data.enums.AbilityScore;
 import com.majinnaibu.monstercards.data.enums.AdvantageType;
 import com.majinnaibu.monstercards.data.enums.ProficiencyType;
 import com.majinnaibu.monstercards.models.Skill;
-import com.majinnaibu.monstercards.ui.shared.ChangeTrackedViewModel;
 import com.majinnaibu.monstercards.utils.ChangeTrackedLiveData;
 
-public class EditSkillViewModel extends ChangeTrackedViewModel {
+public class EditSkillViewModel extends ViewModel {
     private final ChangeTrackedLiveData<AbilityScore> mAbilityScore;
     private final ChangeTrackedLiveData<AdvantageType> mAdvantageType;
+    private final MutableLiveData<Boolean> mHasChanges;
     private final ChangeTrackedLiveData<ProficiencyType> mProficiencyType;
     private final ChangeTrackedLiveData<String> mName;
     private final ChangeTrackedLiveData<Skill> mSkill;
 
     public EditSkillViewModel() {
-        super();
+        mHasChanges = new MutableLiveData<>(false);
-        mAbilityScore = new ChangeTrackedLiveData<>(AbilityScore.STRENGTH, this::makeDirty);
+        ChangeTrackedLiveData.OnValueDirtiedCallback onDirtied = () -> mHasChanges.setValue(true);
-        mAdvantageType = new ChangeTrackedLiveData<>(AdvantageType.NONE, this::makeDirty);
+
-        mProficiencyType = new ChangeTrackedLiveData<>(ProficiencyType.NONE, this::makeDirty);
+        mAbilityScore = new ChangeTrackedLiveData<>(AbilityScore.STRENGTH, onDirtied);
-        mName = new ChangeTrackedLiveData<>("New Skill", this::makeDirty);
+        mAdvantageType = new ChangeTrackedLiveData<>(AdvantageType.NONE, onDirtied);
-        mSkill = new ChangeTrackedLiveData<>(makeSkill(), this::makeDirty);
+        mProficiencyType = new ChangeTrackedLiveData<>(ProficiencyType.NONE, onDirtied);
+        mName = new ChangeTrackedLiveData<>("Unknown Skill", onDirtied);
+        mSkill = new ChangeTrackedLiveData<>(makeSkill(), onDirtied);
     }
 
-    public void copyFromSkill(@NonNull Skill skill) {
+    public EditSkillViewModel(Skill skill) {
+        mHasChanges = new MutableLiveData<>(false);
+        ChangeTrackedLiveData.OnValueDirtiedCallback onDirtied = () -> mHasChanges.setValue(true);
+
+        mAbilityScore = new ChangeTrackedLiveData<>(skill.abilityScore, onDirtied);
+        mAdvantageType = new ChangeTrackedLiveData<>(skill.advantageType, onDirtied);
+        mProficiencyType = new ChangeTrackedLiveData<>(skill.proficiencyType, onDirtied);
+        mName = new ChangeTrackedLiveData<>(skill.name, onDirtied);
+        mSkill = new ChangeTrackedLiveData<>(makeSkill(), onDirtied);
+    }
+
+    public void copyFromSkill(Skill skill) {
         mAbilityScore.resetValue(skill.abilityScore);
         mAdvantageType.resetValue(skill.advantageType);
         mProficiencyType.resetValue(skill.proficiencyType);
         mName.resetValue(skill.name);
-        makeClean();
+    }
+
+    public void copyFromSkill(String name, AbilityScore abilityScore, ProficiencyType proficiency, AdvantageType advantage) {
+        mAbilityScore.resetValue(abilityScore);
+        mAdvantageType.resetValue(advantage);
+        mProficiencyType.resetValue(proficiency);
+        mName.resetValue(name);
     }
 
     public LiveData<Skill> getSkill() {
@@ -74,7 +94,14 @@ public class EditSkillViewModel extends ChangeTrackedViewModel {
         mSkill.setValue(makeSkill());
     }
 
-    @NonNull
+    public LiveData<Boolean> getHasChanges() {
+        return mHasChanges;
+    }
+
+    public boolean hasChanges() {
+        return mHasChanges.getValue();
+    }
+
     private Skill makeSkill() {
         return new Skill(mName.getValue(), mAbilityScore.getValue(), mAdvantageType.getValue(), mProficiencyType.getValue());
     }

Android/app/src/main/java/com/majinnaibu/monstercards/ui/editmonster/EditSkillsFragment.java

@@ -11,6 +11,7 @@ import androidx.fragment.app.Fragment;
 import androidx.lifecycle.ViewModelProvider;
 import androidx.navigation.NavBackStackEntry;
 import androidx.navigation.NavController;
+import androidx.navigation.NavDirections;
 import androidx.navigation.Navigation;
 import androidx.recyclerview.widget.DividerItemDecoration;
 import androidx.recyclerview.widget.ItemTouchHelper;
@@ -32,7 +33,10 @@ public class EditSkillsFragment extends Fragment {
     private ViewHolder mHolder;
 
     private void navigateToEditSkill(Skill skill) {
-        Logger.logUnimplementedFeature("Navigate to the edit skill fragment.");
+        NavDirections action = EditSkillsFragmentDirections.actionEditSkillsFragmentToEditSkillFragment(skill.name, skill.abilityScore, skill.proficiencyType, skill.advantageType);
+        View view = getView();
+        assert view != null;
+        Navigation.findNavController(view).navigate(action);
     }
 
     @Override

Android/app/src/main/res/navigation/mobile_navigation.xml

@@ -127,7 +127,28 @@
             android:id="@+id/editSkillsFragment"
             android:name="com.majinnaibu.monstercards.ui.editmonster.EditSkillsFragment"
             android:label="fragment_edit_skills_list"
-            tools:layout="@layout/fragment_edit_skills_list" />
+            tools:layout="@layout/fragment_edit_skills_list">
+            <action
+                android:id="@+id/action_editSkillsFragment_to_editSkillFragment"
+                app:destination="@id/editSkillFragment" />
+        </fragment>
+        <fragment
+            android:id="@+id/editSkillFragment"
+            android:name="com.majinnaibu.monstercards.ui.editmonster.EditSkillFragment"
+            android:label="fragment_edit_skill"
+            tools:layout="@layout/fragment_edit_skill">
+            <argument
+                android:name="name"
+                app:argType="string" />
+            <argument
+                android:name="abilityScore"
+                app:argType="com.majinnaibu.monstercards.data.enums.AbilityScore" />
+            <argument
+                android:name="proficiency"
+                app:argType="com.majinnaibu.monstercards.data.enums.ProficiencyType" />
+            <argument
+                android:name="advantage"
+                app:argType="com.majinnaibu.monstercards.data.enums.AdvantageType" />
+        </fragment>
     </navigation>
-
+</navigation>
-</navigation>