-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: profile card #1319
base: ui/redesign
Are you sure you want to change the base?
feat: profile card #1319
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## ui/redesign #1319 +/- ##
===========================================
Coverage 12% 12%
===========================================
Files 265 265
Lines 7209 7209
===========================================
Hits 806 806
Misses 6403 6403 |
Add tooltip on long click in both deadline and print balance cards; |
5626b5a
to
55d23cd
Compare
Do not forget to change Generic Card |
@vitormpp Fix the conflicts please |
987e671
to
1fc65dc
Compare
1fc65dc
to
429b869
Compare
429b869
to
1e4d072
Compare
padding: padding ?? const EdgeInsets.all(10), child: child), | ||
padding: padding ?? const EdgeInsets.all(10), | ||
child: GestureDetector( | ||
onTap: onClick != null ? () => onClick!() : null, |
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.
can't this just be onTap: onClick
?
Closes #1309
Generic Card
Profile Card