Skip to content

Conversation

ndonkoHenri
Copy link
Contributor

@ndonkoHenri ndonkoHenri commented May 3, 2024

Make code easier to use/reuse especially for new contributors.

  • Cleanup code
  • Use/create utility methods where possible

(Part has been done in #3175 and #3263)

@ndonkoHenri ndonkoHenri mentioned this pull request May 10, 2024
3 tasks
@ndonkoHenri ndonkoHenri marked this pull request as ready for review May 14, 2024 16:20
@ndonkoHenri ndonkoHenri requested a review from FeodorFitsner May 14, 2024 16:20
@FeodorFitsner
Copy link
Contributor

Please resolve merging conflict.

# Conflicts:
#	packages/flet/lib/src/controls/safe_area.dart
#	packages/flet/lib/src/utils/colors.dart
#	packages/flet/lib/src/utils/text.dart
@FeodorFitsner
Copy link
Contributor

Conflict again.

# Conflicts:
#	packages/flet/lib/src/controls/chip.dart
#	packages/flet/lib/src/controls/datatable.dart
#	packages/flet/lib/src/controls/segmented_button.dart
#	sdk/python/packages/flet-core/src/flet_core/icon_button.py
Copy link
Contributor

@FeodorFitsner FeodorFitsner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great spring cleanup! 🧹

@FeodorFitsner FeodorFitsner merged commit 8e3e441 into main May 24, 2024
@ndonkoHenri ndonkoHenri deleted the code-refactor branch May 25, 2024 13:20
zrr1999 pushed a commit to zrr1999/flet that referenced this pull request Jul 17, 2024
* Enhance typing, use _set_enum_attr

* Fix failing tests concerning TextOverflow.NONE

* parseClip util

* Filter children using isVisible

* refactor parseCurve util

* refactor parseTextAlign util

* create parseTextOverflow util

* create parseTextCapitalization util

* create parseDismissDirection util

* create parseOrientation util

* refactor parseEdgeInsets util to accept defaultValue

* refactor parseWrapAlignment util

* refactor parseTabAlignment util

* refactor parseWrapCrossAlignment util

* refactor parseCrossAxisAlignment util

* refactor parseMainAxisAlignment util

* refactor parseAnimation util to accept defaultValue

* refactor parseBorderRadius util to accept defaultValue

* refactor parseRadius util to accept defaultValue

* refactor parseTextInputType util to accept defaultValue

* refactor parseBlur util to accept defaultValue

* refactor parseImageRepeat util to accept defaultValue

* refactor parseBoxFit util to accept defaultValue

* create parseFilterQuality

* refactor parseMouseCursor util: set null as default

* refactor parseBrightness util

* refactor parseRotate, parseScale, parseOffset utils to accept defaultValue

* create parseStrokeCap and parseStrokeJoin

* lowercase cursor values

* create parseColor

* refactor parseVisualDensity

* refactor edge_insets.dart

* refactor parseAlignment, alignmentFromJson

* refactor parseIcon

* refactor parseTileMode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants