mnojek commented on a change in pull request #18715:
URL: https://github.com/apache/airflow/pull/18715#discussion_r722925502
##########
File path: airflow/www/templates/airflow/confirm.html
##########
@@ -21,7 +21,7 @@
{% block content %}
{{ super() }}
- <h2>Wait a minute.</h2>
+ <h2>Wait a minute</h2>
Review comment:
This dot just doesn't look nice in the UI. Short sentences or
instructions don't need a dot at the end, especially if they are not included
in some bigger text wall, but are just a short notification or instruction. I
know it's subjective, but I think it's commonly accepted. Sentences like 'Click
OK', 'Check your filters', 'Insert value' do not require a full stop at the
end. Also, it is perceived as insincere or even rude nowadays. In my opinion,
it looks just better in this case.
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
