Copilot commented on code in PR #1359:
URL: https://github.com/apache/airflow-site/pull/1359#discussion_r2649900059
##########
sphinx_airflow_theme/sphinx_airflow_theme/layout.html:
##########
@@ -387,10 +387,18 @@
document.querySelectorAll('[data-bs-theme-value]').forEach(element => {
element.classList.remove('active')
element.setAttribute('aria-pressed', 'false')
+ const checkIcon = element.querySelector('.bi.ms-auto')
+ if (checkIcon) {
+ checkIcon.classList.add('d-none')
+ }
})
btnToActive.classList.add('active')
btnToActive.setAttribute('aria-pressed', 'true')
+ const activeCheckIcon = btnToActive.querySelector('.bi.ms-auto')
+ if (activeCheckIcon) {
+ activeCheckIcon.classList.remove('d-none')
+ }
Review Comment:
Inconsistent indentation detected. The code uses 12 spaces for the inner if
statement, which is inconsistent with the rest of the file that uses 6 spaces
per indentation level. This should be corrected to maintain consistent code
formatting throughout the file.
##########
sphinx_airflow_theme/sphinx_airflow_theme/layout.html:
##########
@@ -387,10 +387,18 @@
document.querySelectorAll('[data-bs-theme-value]').forEach(element => {
element.classList.remove('active')
element.setAttribute('aria-pressed', 'false')
+ const checkIcon = element.querySelector('.bi.ms-auto')
+ if (checkIcon) {
+ checkIcon.classList.add('d-none')
+ }
Review Comment:
Inconsistent indentation detected. The code uses 4 spaces for the outer
block but 12 spaces (3 levels) for the inner if statement, which is
inconsistent with the rest of the file that uses 6 spaces per indentation
level. This should be corrected to maintain consistent code formatting
throughout the file.
--
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]
