-
Notifications
You must be signed in to change notification settings - Fork 36
feat: 在平板或PC视图下考勤查询功能使用表格视图 #109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR enhances the attendance query feature by introducing a responsive table view for tablets and PCs (screens wider than 800px) while maintaining the existing card view for mobile devices. The table view includes sorting and filtering capabilities, and the detail navigation has been refactored to use BothSideSheet instead of context.push() for a more consistent user experience.
Key changes:
- Added responsive layout switching based on screen width (800px threshold)
- Implemented table view with sortable columns and status-based filtering
- Refactored detail view navigation to use
BothSideSheetfor better UX - Added internationalization support for table-related UI elements
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| pubspec.lock | Multiple package version downgrades including flex_color_scheme, flex_seed_scheme, meta, test_api, and Flutter SDK minimum version |
| lib/page/public_widget/both_side_sheet.dart | Enhanced title handling with overflow protection using Expanded widget and ellipsis |
| lib/page/class_attendance/class_attendance_view.dart | Added responsive layout logic to switch between table and card views based on screen width |
| lib/page/class_attendance/class_attendance_table.dart | New file implementing table view with sorting, filtering, and status-based display features |
| lib/page/class_attendance/class_attendance_detail.dart | Refactored to support conditional AppBar rendering for BothSideSheet integration |
| lib/page/class_attendance/class_attandance_card.dart | Updated to use BothSideSheet instead of context.push() for detail navigation |
| assets/flutter_i18n/zh_TW.yaml | Added Traditional Chinese translations for table view UI elements |
| assets/flutter_i18n/zh_CN.yaml | Added Simplified Chinese translations for table view UI elements |
| assets/flutter_i18n/en_US.yaml | Added English translations for table view UI elements |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
好设计,要是早点发就好了,我新版本刚发布:P 关于未在ios平台上测试:该功能仅涉及 Flutter 代码,这块是跨平台的,在安卓手机和平板上运行 OK 就行了。 我考虑到平板等平台可能涉及的窗口变化导致的变化,可以试试把这个应用在桌面(Windows 或者 Linux 环境下)来运行,变换窗口大小。 |
感谢作者大大,新功能在Windiws和Android模拟器环境下未见异常,但是在快速拖动窗口边框重置大小时 另外,问一下作者是否有考虑为项目构建一个正式的主页,如果有需要我可以在寒假期间用next.js搭一个。 |
这块我再审阅,最近我在忙别的事项,代码还没看:P 从描述来说,应该使用 LayoutBuilder 来搞在视图大小变化中更改布局的需求。
QQ 联系,加 PDA 交流群找我。我是想,但是最终的成品可能在设计上很简单…… |
这个PR实现了考勤查询功能的界面更新。
context.push()更改为与课表类似的BothSideSheetNote
由于我没有mac设备,新功能未在ios平台上进行测试。
Preview: