-
-
Notifications
You must be signed in to change notification settings - Fork 329
added study filter #2369
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?
added study filter #2369
Conversation
|
Thanks for this. Can you please post a screenshot, or better, a screen recording to show the difference? |
Record_2025-11-19-12-18-49.mp4I want to, but am unable to login for some reason. Maybe some setup I didn't configure. |
|
And the option for filter is only for logged in user. |
You can make an account on lichess.dev instead and use that do demonstrate the change |
VID-20251120-WA0014.mp4Feel free to tell about any changes here. |
|
Hello @tom-anders , any updates here? |
| collection: ^1.17.0 | ||
| connectivity_plus: ^7.0.0 | ||
| cronet_http: ^1.3.1 | ||
| cronet_http: 1.6.0 |
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.
Why this change? It has nothing to do with the PR. Please remove it (also it fixes the version which we don't want).
| return PlatformScaffold( | ||
| appBar: PlatformAppBar( | ||
| title: Text(sessionUser != null ? context.l10n.studyMenu : context.l10n.studyAllStudies), | ||
| title: Text( |
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.
You should use the AppBarTitleText widget here, as it will auto size the text to fit in.
I've added the study filter as needed in the issue .