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

Links between SPA apps should not open a new tab #7480

Closed
indirectlylit opened this issue Aug 18, 2020 · 1 comment
Closed

Links between SPA apps should not open a new tab #7480

indirectlylit opened this issue Aug 18, 2020 · 1 comment
Assignees
Labels
P0 - critical Priority: Release blocker or regression TAG: design system Change to our style guide / design system TAG: regression Something that previously worked
Milestone

Comments

@indirectlylit
Copy link
Contributor

indirectlylit commented Aug 18, 2020

Observed behavior

Navigate between facility and device settings pages Explore without account
2020-08-17 17 45 32 2020-08-17 17 55 44

Related:

First noticed because in learningequality/kolibri-installer-gnome#7 for the GTK/webkit Gnome app, special handling had to be added so that "external links" which were actually internal links would be opened in a webkit window.

Expected behavior

From a user's perspective, the fact that different parts of Kolibri are implemented as separate SPAs is an implementation detail, not a feature. Navigating from one to the other should keep them in the same window.

  • We should have a way of controlling whether or not links get opened in a new window
  • Links that do get opened in a new window should include an icon, similar to that used in the design system: image
  • We should audit use of KExternalLink and ensure that cross-SPA links do not open in a new window

User-facing consequences

users are opening new tabs when they should be staying in the same tab

Steps to reproduce

Click 'explore without account'

Context

0.14.1

@indirectlylit indirectlylit added P1 - important Priority: High impact on UX TAG: regression Something that previously worked DEV: frontend TAG: design system Change to our style guide / design system labels Aug 18, 2020
@indirectlylit indirectlylit added this to the upcoming patch milestone Aug 18, 2020
@indirectlylit indirectlylit added P0 - critical Priority: Release blocker or regression and removed P1 - important Priority: High impact on UX labels Aug 18, 2020
@indirectlylit indirectlylit modified the milestones: upcoming patch, 0.14.2 Aug 18, 2020
@nucleogenesis
Copy link
Member

We should audit use of KExternalLink and ensure that cross-SPA links do not open in a new window

Bah I thought I did this :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 - critical Priority: Release blocker or regression TAG: design system Change to our style guide / design system TAG: regression Something that previously worked
Projects
None yet
Development

No branches or pull requests

3 participants