Skip to content
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

Fix Gray Background Issue on Error in CustomDropdown #94

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mreslamgeek
Copy link

Bug Fixes

CustomDropdown Error Background

  • Resolved an issue where a gray background would appear when an error message was displayed in the CustomDropdown widget.
  • The InputDecorator now sets the fillColor to Colors.transparent when an error occurs, ensuring a consistent and clean appearance.
  • This change helps maintain the visual integrity of the dropdown, especially in forms where error messages are displayed.
image

   - Updated the  class to handle error states more effectively.
   - Set  to  in  when an error occurs, preventing an unwanted gray background.
   - Enabled the  property to ensure the  is applied correctly.
   - Ensured compatibility with existing custom decorations by maintaining the default error style.

   This change addresses the issue where a gray background would appear when an error message was displayed in the  widget, providing a consistent and clean appearance.
- Set  to  in  when an error occurs.
- Enabled  property to ensure the  is applied.
@mreslamgeek
Copy link
Author

the updated version
image

Copy link

@codemenriyad codemenriyad left a comment

Choose a reason for hiding this comment

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

Why this pr is not merged, i face this issue i need this.

@mreslamgeek
Copy link
Author

@codemenriyad you can use my own version until @AbdullahChauhan accept the pull request

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

Successfully merging this pull request may close these issues.

2 participants