-
Notifications
You must be signed in to change notification settings - Fork 466
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
DropdownMenu
Control
#3638
base: main
Are you sure you want to change the base?
DropdownMenu
Control
#3638
Conversation
Reviewer's Guide by SourceryThis pull request introduces a new File-Level Changes
Tips
|
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.
Hey @ndonkoHenri - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟡 Complexity: 1 issue found
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.
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.
Hey @ndonkoHenri - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 13 issues found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟡 Complexity: 1 issue found
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
sdk/python/packages/flet-core/src/flet_core/form_field_control.py
Outdated
Show resolved
Hide resolved
# Conflicts: # sdk/python/packages/flet-core/src/flet_core/dismissible.py # sdk/python/packages/flet-core/src/flet_core/dropdown.py # sdk/python/packages/flet-core/src/flet_core/navigation_bar.py # sdk/python/packages/flet-core/src/flet_core/page.py # sdk/python/packages/flet-core/src/flet_core/scrollable_control.py # sdk/python/packages/flet-core/src/flet_core/snack_bar.py # sdk/python/packages/flet-core/src/flet_core/tabs.py # sdk/python/packages/flet-core/src/flet_core/textfield.py # sdk/python/packages/flet-core/src/flet_core/time_picker.py # sdk/python/packages/flet-core/src/flet_core/types.py
Could you resolve a conflict in this PR please? |
# Conflicts: # packages/flet/lib/src/controls/dropdown.dart # packages/flet/lib/src/controls/textfield.dart # packages/flet/lib/src/utils/form_field.dart # sdk/python/packages/flet-core/src/flet_core/dropdown.py # sdk/python/packages/flet-core/src/flet_core/form_field_control.py # sdk/python/packages/flet-core/src/flet_core/textfield.py # sdk/python/packages/flet-core/src/flet_core/types.py
I've tried the new control and observed the strange behavior - it triggers changed event on the second click. Here's the video: Screen.Recording.2024-08-08.at.10.58.41.AM.movDo you have the same issue? I built macOS client with Flutter 3.24.0. |
Merging gone wrong? |
Closes #1088
Test Code
Summary by Sourcery
This pull request introduces a new
DropdownMenu
control with extensive customization options and enhances theFormFieldControl
to support additional properties for better form field customization. It also includes a bug fix for a typo in theAnimationCurve
enum.DropdownMenu
control with various customization options including prefix, suffix, helper, and error controls, as well as icon colors and text styles.AnimationCurve
enum fromFAST_OUT_SLOWIN
toFAST_OUT_SLOW_IN
.FormFieldControl
to support additional properties such asicon_color
,prefix_icon_color
,suffix_icon_color
,focus_color
,align_label_with_hint
,floating_label_text_style
,active_indicator_border_side
,hint_fade_duration
,error_max_lines
,helper_max_lines
, andhint_max_lines
.InputDecoration
andInputDecorationTheme
to include new properties for better customization of form fields.Dropdown
,TextField
, andDropdownMenu
controls to utilize the new properties and enhancements inFormFieldControl
.