potiuk commented on PR #23311: URL: https://github.com/apache/airflow/pull/23311#issuecomment-1112355360
> Thank you for tagging me!! I love PRs like this. I don't know a ton about accessibility + CLIs but I went down a rabbit hole looking to see if bold/italic/underline is "best practice" (imho it's a great idea and is better than solely using color but I was curious what other CLIs do, and I'm not an expert so I don't want to be the deciding voice here) - I didn't find anything about that but I did find [this study](https://dl.acm.org/doi/fullHtml/10.1145/3411764.3445544) from the ACM and it reinforces the fact that being able to turn off the ASCII art is an excellent idea Oh yeah. I personally like it but I know it's noise for most people :). Also there is an interesting point which breeze is mostly following (but I have not thought about accessibility): > Recommendation 5: Ensure that all commands provide status and progress indication. Especially "success" message is an important one because indeed when the command silently exists you might not realised it did so. I think in vast majority of cases we already have it (and `[success]` style is specifically foreseen for that) but I will pay an extra attention to it :). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
