search

doctor-g / ThunderstoneQuestRandomizer

Flutter implementation of a Thunderstone Quest Randomizer
GNU General Public License v3.0
5 stars 0 forks source link

Replace manual checkbox labeling with CheckboxListTile #5

Closed doctor-g closed 3 years ago

doctor-g commented 3 years ago

Right now, the settings screen manually connects a Checkbox with a TextButton. Reviewing the documentation reveals that CheckboxListTile does exactly that, so it should be used instead.

doctor-g commented 3 years ago

Looking more into this, CheckboxListStyle always puts the checkbox on the right, but this would be awkward for specifying the quests at the very least. A better approach is explained by the CheckboxListStyle documentation: to make a custom combination of widgets to show the checkbox on the left and text on the right. This will be functionally equivalent to the current code, but it would remove some of the unnecessary redundancy.