-
Notifications
You must be signed in to change notification settings - Fork 443
Add MultilineTextBox component #1618
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
Comments
I'm not sure if we want to increase the complexity of the TextBox class beyond how complex it is already. It should likely be refactored before attempting this. I'd have to see a sample implementation to decide whether it would be part of
The ones you specify would be a fine start.
Yes. It will be used for editing longer articles like forum posts. |
Could you elaborate on this? Is there something wrong with the |
I thin splitting individual visual components into their own classes would be a good start, if that isn't done yet. ie. |
I'll work on this. |
Unassigning myself for now as it is a low priority from being worked on. |
Uh oh!
There was an error while loading. Please reload this page.
As I need this and was tinkering with it I thought this is an opportunity to open an issue to ask questions before I open a PR in the coming days. My tinkering consisted of copying the entire
TextBox
and see if it's feasible with my knowledge to do this.I managed
I need to figure out
Question time:
TextBox
or is it just an option you can enable for the usualTextBox
? I guess it'll be the former.TextBox
could use as well maybe?TextBox
with many lines need that I just overlooked?Of course this won't ask/answer everything as code should be available for that but something fundamental (like the first question) may already be answered beforehand.
The text was updated successfully, but these errors were encountered: