blockbasti / just_another_workout_timer

A simple timer for your workouts, built with Flutter!
MIT License
127 stars 21 forks source link

Named Sets and toggle for Set exercise visiblity #201

Open crimsoncor opened 2 months ago

crimsoncor commented 2 months ago

A couple small quality of life changes as I learn dart and flutter.

1) Add a way to give a name to a set. Currently this is triggered via a long-press on the ListTile which contains the set name using a flutter prompt dialog. I considered making the set name a live text field, like the workout name, but the set editing interface is already busy and I didn't want to add something else that a user could accidentally click on. I don't love that the edit facility is now not easily discoverable, though. I played with adding an edit button but no matter where I put it, it really messed with the layout of the ListTile. Open to other and better suggestions for this.

Set names are saved to json and show up in the workout runner. I limited them to 15 characters, but even at that length, they can push the timer in the workout builder onto a new line, so I just permanent split the set name and the timer onto separate lines. I also dropped the font size slightly on the set name.

2) Add a way to hide the exercise list for a set in the workout builder. Makes it much easier to re-ordered sets or to just concentrate on the current set being built. Since this is just UI sugar, I did not persist it to JSON. I also purposely did not copy it when duplicating a set. This way, you can use a workflow where you make a set, collapse its exercises, duplicate it, and then immediately be ready to edit the new set. This also necessitated fixing a small bug where a duplicated set was being inserted before the current set. It wasn't noticeable before sets could be collapsed, but it is now.

crimsoncor commented 2 months ago

Screen shots of the new features. Screenshot_20240425_110535 Screenshot_20240425_111640 Screenshot_20240425_111706 Screenshot_20240425_111720 Screenshot_20240425_111745