sympa-community / sympa

Sympa, Mailing List Management Software
https://www.sympa.community/sympa
GNU General Public License v2.0
243 stars 96 forks source link

Add autofocus to input text web #1648

Closed dominiquefournier closed 1 year ago

ikedas commented 1 year ago

Hi @dominiquefournier ,

Could you please combine multiple PRs into one?


BTW, IMHO we'd be better to consider accessibility. Is there a good way to deal with this?

dominiquefournier commented 1 year ago

@ikedas I don't know how combine multiple PR into one. Do you have a documentation ?

ikedas commented 1 year ago

You'd be better to re-submit a new PR including multiple commits.

ikedas commented 1 year ago

Additionally, how about adding autofocus attribute to these as well?

ikedas commented 1 year ago

@dominiquefournier , I opened a new issue #1686. Once you have learned how to submit a PR with multiple changes, please submit another PR to fix this issue. This PR will be closed for now.