Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix external links to open in new tab #140

Merged
merged 3 commits into from
Aug 7, 2024

Conversation

jiya-singhal
Copy link
Contributor

@jiya-singhal jiya-singhal commented Aug 6, 2024

resolved issue #139

Description:
This pull request updates the external links on the bottom of the page to open in a new tab. The change involves adding the target="_blank" attribute to anchor tags linking to GitHub, the App Store, the Play Store, and the Open Transit Software Foundation. This enhancement ensures that when users click on these external links, they are directed to the respective sites in a new browser tab, preserving their current session on the application.

Changes Made:

Added target="_blank" to the GitHub link.
Added target="_blank" to the App Store link.
Added target="_blank" to the Play Store link.
Added target="_blank" to the Open Transit Software Foundation link.
Motivation:
Opening external links in a new tab prevents users from leaving the current application session, providing a better user experience by maintaining the context of the original page.

Testing:
The changes were manually tested to verify that all external links now open in a new tab. No other parts of the application were affected by these changes.

Checklist:

Added target="_blank" to external links
Manually tested the changes
Verified that the change does not impact other parts of the application

Screencast.from.06-08-24.09.37.47.PM.IST.webm

@CLAassistant
Copy link

CLAassistant commented Aug 6, 2024

CLA assistant check
All committers have signed the CLA.

@jiya-singhal
Copy link
Contributor Author

Respected projects maintainers

I've updated the external links on the bottom of the page to open in a new tab, as discussed. The changes include adding target="_blank" to the relevant links to ensure users are not navigated away from the application.

Could someone please review the changes and provide feedback? Your input would be greatly appreciated.

Thank you!

Copy link
Member

@aaronbrethorst aaronbrethorst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you! Before I can merge this, though, I will need you to please revert all changes that are unrelated to the issue you're resolving.

@jiya-singhal
Copy link
Contributor Author

Hi @aaronbrethorst,

As requested, I have reverted the unrelated changes. The pull request now focuses solely on the issue of external links opening in a new tab.
Could you please review the updated code?

Thank you!

Copy link
Member

@aaronbrethorst aaronbrethorst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, thank you!

@aaronbrethorst aaronbrethorst merged commit aabddf0 into OneBusAway:main Aug 7, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants