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

feat: Add customizable border color for text input component #4191

Merged
merged 3 commits into from Oct 26, 2022
Merged

feat: Add customizable border color for text input component #4191

merged 3 commits into from Oct 26, 2022

Conversation

bsantosh909
Copy link
Contributor

@bsantosh909 bsantosh909 commented Oct 1, 2022

fixes: #4165

Example:
Screen Shot 2022-10-01 at 1 34 05 PM

NOTE: The right most input is the default text input without any border color defined.

@bsantosh909
Copy link
Contributor Author

bsantosh909 commented Oct 1, 2022

NOTE: Having no default value for the border color seems to mess up the design a bit, so if you guys could provide the desired default value as the border, I can add it 😄
Screen Shot 2022-10-01 at 1 37 42 PM

@arpitnath arpitnath added the run-ci CI is run only when this label is added label Oct 4, 2022
@arpitnath
Copy link
Collaborator

@TheLearneer Yeah, we can have the existing border color as default value, e.g., #dadcde

@bsantosh909
Copy link
Contributor Author

Hi @arpitnath Just in case you missed it, the changes have been made 🙏

@gsmithun4
Copy link
Collaborator

deployed https://tj-4191.herokuapp.com/

@kavinvenkatachalam kavinvenkatachalam merged commit 0c895e2 into ToolJet:develop Oct 26, 2022
@bsantosh909 bsantosh909 deleted the style-4165/text-input-border-color branch October 26, 2022 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted run-ci CI is run only when this label is added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposal to add border color option to Text Input widget
4 participants