-
Notifications
You must be signed in to change notification settings - Fork 253
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
Improve SuperTextField demo app #1598
Comments
Hi @matthew-carroll I would like to contribute to this. I will create a PR for this soon. |
I am a little confused about this point:
Please correct me if I am wrong. Should I add a + button to add multiple |
I have pushed 3/4 tasks mentioned in the issue above. Please let me know about the following query & I will add that in the PR.
cc - @matthew-carroll |
Hi @foxtrotravi thanks for your desire to contribute. This ticket is meant for me - it's a note to myself. I see you've done some good work assembling text field variations. I appreciate that effort. But it's not quite what I had in mind. If you really want to work on this, I can put together a few design mockups. Let me know if that's something you want to do. In general, you should wait for a ticket to be assigned before working on it. We have all sorts of things going on behind the scenes that might interfere with your ability to implement a ticket. I'm not sure we have any other tickets right now that have a clear goal for new project developers. |
Hi @matthew-carroll Yeah, I totally understand that I should have waited for the ticket to be assigned. |
We added an entrypoint specifically to demo
SuperTextField
in this PR: #1596That work is a start, but we can create a more compelling demo app for
SuperTextField
.Implement the following:
SuperTextField
sSuperTextFields
in a vertical column.SuperTextField
and alter various characteristics in a settings pane, e.g., line count, border color, etc.The text was updated successfully, but these errors were encountered: