Architecture #10
Architecture #10
Conversation
ccitygril
commented
Jul 24, 2024
ccitygril
commented
- сделана верста
- добавлен Elementary
- проведен небольшой рефактор
- в процессе di
There was a problem hiding this comment.
В целом выглядит круто. Только по поводу структуры проекта - у тебя почти в папку ui грубо говоря попали фичи. Лучше сделать всё-таки папку features в lib, где у тебя будут лежать theme и image, в image перенести все папки data, di и т.д. из lib и создать папку screens, где уже разместить свои elementary-экраны
| @freezed | ||
| class ImageEntity with _$ImageEntity { | ||
| const factory ImageEntity({ | ||
| required int id, |
There was a problem hiding this comment.
Скорее всего id у нас будут строки, а точнее пути до файлов
| @@ -0,0 +1,5 @@ | |||
| import 'package:flutter/material.dart'; | |||
|
|
|||
| class AppColor { | |||
| import 'package:flutter/material.dart'; | ||
| import 'package:surf_flutter_summer_school_24/ui/ui_kit/color/color_theme.dart'; | ||
|
|
||
| abstract class AppThemeData { |
There was a problem hiding this comment.
Оно всё-таки должно быть в uikit
| } | ||
| } | ||
|
|
||
| class PageCarousel extends StatefulWidget { |
There was a problem hiding this comment.
Не очень понимаю, а это точно что-то нужное?
| @override | ||
| void initWidgetModel() { | ||
| super.initWidgetModel(); | ||
| } | ||
| } |
There was a problem hiding this comment.
Если пустой, то не надо переопределять метод
There was a problem hiding this comment.
А, ты сюда подгрузку планируешь добавить. Тогда ок
| Widget build(BuildContext context) { | ||
| return Scaffold( | ||
| appBar: const CustomAppBar(), | ||
| body: Column( |
| ElevatedButton( | ||
| onPressed: () {}, child: const Text("ПОПРОБОВАТЬ СНОВА")) |
There was a problem hiding this comment.
Кнопочку лучше вынести в отдельный виджет
|
Только сюда не сливай, сливай в свой репозиторий |
