You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "dheerajturaga (via GitHub)" <gi...@apache.org> on 2025/10/03 04:35:54 UTC
[PR] Update Airflow SDK Param text input to use textarea with text wrapping [airflow]
dheerajturaga opened a new pull request, #56354:
URL: https://github.com/apache/airflow/pull/56354
Long input textstrings need to wrap so that it is easy to view the contents
Changed FieldString component from Input to Textarea to enable text
wrapping for better user experience when entering longer string values.
Removed datalist support as it's incompatible with textarea elements.
Before:
<img width="1093" height="718" alt="image" src="https://github.com/user-attachments/assets/8d57fa38-0fb9-4769-8703-668b2c22c6b6" />
After:
<img width="1108" height="736" alt="image" src="https://github.com/user-attachments/assets/0117afa5-daeb-45a6-a322-1671215c214e" />
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Update Airflow SDK Param text input to use textarea with text wrapping [airflow]
Posted by "jscheffl (via GitHub)" <gi...@apache.org>.
jscheffl commented on PR #56354:
URL: https://github.com/apache/airflow/pull/56354#issuecomment-3365176510
> Ah I see, thanks for pointing me to this! are you suggesting multiline be the default ?
No I was proposing to keep it as most users can use single line and if you assume you will have larger input then you can add the multiline attribute wherever needed. Basically I'd opt for keeping it as is. Except there is a real reason to change default
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Update Airflow SDK Param text input to use textarea with text wrapping [airflow]
Posted by "dheerajturaga (via GitHub)" <gi...@apache.org>.
dheerajturaga commented on PR #56354:
URL: https://github.com/apache/airflow/pull/56354#issuecomment-3365160886
Ah I see, thanks for pointing me to this! are you suggesting multiline be the default ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Update Airflow SDK Param text input to use textarea with text wrapping [airflow]
Posted by "dheerajturaga (via GitHub)" <gi...@apache.org>.
dheerajturaga closed pull request #56354: Update Airflow SDK Param text input to use textarea with text wrapping
URL: https://github.com/apache/airflow/pull/56354
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Update Airflow SDK Param text input to use textarea with text wrapping [airflow]
Posted by "dheerajturaga (via GitHub)" <gi...@apache.org>.
dheerajturaga commented on PR #56354:
URL: https://github.com/apache/airflow/pull/56354#issuecomment-3365181668
> > Ah I see, thanks for pointing me to this! are you suggesting multiline be the default ?
>
>
>
> No I was proposing to keep it as most users can use single line and if you assume you will have larger input then you can add the multiline attribute wherever needed. Basically I'd opt for keeping it as is. Except there is a real reason to change default
Agreed! Will close this
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org