You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "versha890 (via GitHub)" <gi...@apache.org> on 2025/12/12 18:40:10 UTC
[PR] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
versha890 opened a new pull request, #1311:
URL: https://github.com/apache/airflow-site/pull/1311
This PR fixes the unintuitive behavior of the "Previous" and "Next" buttons on the Use Cases pages of the Airflow website.
**Problem:**
- "Previous" navigated forward (to the next item)
- "Next" navigated backward (to the previous item)
**Solution:**
- Swapped `.PrevInSection` and `.NextInSection` in `layouts/partials/pager.html`
- Now "Next" → right, "Previous" → left, providing intuitive navigation
**Testing:**
1. Run `hugo server -D` locally
2. Open http://localhost:1313/use-cases/etl_analytics/
3. Verify buttons navigate correctly
--
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] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
Posted by "versha890 (via GitHub)" <gi...@apache.org>.
versha890 closed pull request #1311: Fix: Swap Previous/Next buttons for intuitive navigation in use cases
URL: https://github.com/apache/airflow-site/pull/1311
--
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] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on code in PR #1311:
URL: https://github.com/apache/airflow-site/pull/1311#discussion_r2615298682
##########
package-lock.json:
##########
@@ -0,0 +1,6 @@
+{
Review Comment:
this is accidental I guess?
--
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] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #1311:
URL: https://github.com/apache/airflow-site/pull/1311#issuecomment-3647958882
First fix the static check issues.
--
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] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #1311:
URL: https://github.com/apache/airflow-site/pull/1311#issuecomment-3647963694
And no - the change to package.lock has not been reverted - if what you copy&paste is what AI agent tells you - I recommend you actually review the changes before submitting it (actually you have not submitted the change yet)
--
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] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #1311:
URL: https://github.com/apache/airflow-site/pull/1311#issuecomment-3647793891
Indded not intuitive.
--
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] Fix: Swap Previous/Next buttons for intuitive navigation in use cases [airflow-site]
Posted by "versha890 (via GitHub)" <gi...@apache.org>.
versha890 commented on PR #1311:
URL: https://github.com/apache/airflow-site/pull/1311#issuecomment-3647867006
Hi @potiuk,
I have reverted the accidental `package-lock.json` changes in this branch.
The PR now only contains the intended fix for the "Previous" and "Next" buttons.
Please review again when you get a chance. Thanks! 🙏
--
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