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

[E & A] Links to catalog should not exit E&A #861

Closed
1 task
j08lue opened this issue Feb 28, 2024 · 0 comments · Fixed by #863
Closed
1 task

[E & A] Links to catalog should not exit E&A #861

j08lue opened this issue Feb 28, 2024 · 0 comments · Fixed by #863
Assignees

Comments

@j08lue
Copy link
Contributor

j08lue commented Feb 28, 2024

We want to give users an option to go to the data catalog page about a dataset, but they should not accidentally exit E&A (and loose their state there).

Two places, links to the catalog currently exit E&A. Instead, we should have them open in new tabs.

The text link in the dataset selector modal header

image

<ModalIntro>
<p>This tool allows the exploration and analysis of time-series datasets in raster format. For a comprehensive list of available datasets, please visit the <Link to={DATASETS_PATH}>Data Catalog</Link>.
</p>
</ModalIntro>

The button on the layer info modal:

image

footerContent={
<ButtonStyleLink to={dataCatalogPage} onClick={close} variation='primary-fill' size='medium'>
Open in Data Catalog
</ButtonStyleLink>

Acceptance criteria

  • No links from E&A to data catalog exit E&A. Instead, they open in new tabs.
@j08lue j08lue added this to the E&A Revision 3 milestone Feb 28, 2024
@sandrahoang686 sandrahoang686 self-assigned this Feb 28, 2024
sandrahoang686 added a commit that referenced this issue Feb 29, 2024
This PR updates the data catalog links for A&E to open in a new tab
instead of the exiting current screen.

Closes #861
Addresses #856
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 a pull request may close this issue.

2 participants