Skip to content

Commit

Permalink
Undid SliverList builder method extraction
Browse files Browse the repository at this point in the history
  • Loading branch information
mitryp committed May 25, 2023
1 parent 47056f9 commit d6c2903
Showing 1 changed file with 15 additions and 18 deletions.
33 changes: 15 additions & 18 deletions lib/utils/mistake_popup.dart
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ class LanguageToolMistakePopup extends StatelessWidget {
const _borderRadius = 10.0;
const _mistakeNameFontSize = 13.0;
const _mistakeMessageFontSize = 15.0;
const _replacementButtonsSpacing = 10.0;

const padding = 10.0;
const paddingCount = 4;
Expand Down Expand Up @@ -120,30 +121,26 @@ class LanguageToolMistakePopup extends StatelessWidget {
),
SliverList.builder(
itemCount: mistake.replacements.length,
itemBuilder: _suggestionsListBuilder,
itemBuilder: (context, index) {
final replacement = mistake.replacements[index];

return Padding(
padding: const EdgeInsets.all(_replacementButtonsSpacing / 2),
child: ElevatedButton(
onPressed: () {
controller.replaceMistake(mistake, replacement);
popupRenderer.dismiss();
},
child: Text(replacement),
),
);
},
),
],
),
);
}

Widget _suggestionsListBuilder(BuildContext _, int index) {
const replacementButtonsSpacing = 10.0;

final replacement = mistake.replacements[index];

return Padding(
padding: const EdgeInsets.all(replacementButtonsSpacing / 2),
child: ElevatedButton(
onPressed: () {
controller.replaceMistake(mistake, replacement);
popupRenderer.dismiss();
},
child: Text(replacement),
),
);
}

double _calculateAvailableSpace(
BuildContext context, {
required double paddings,
Expand Down

0 comments on commit d6c2903

Please sign in to comment.