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

Variant icons leads to the wrong postion on the variant details page #1315

Closed
flokraft85 opened this issue Jan 31, 2024 · 5 comments · Fixed by #1465
Closed

Variant icons leads to the wrong postion on the variant details page #1315

flokraft85 opened this issue Jan 31, 2024 · 5 comments · Fixed by #1465
Assignees
Labels
bug Something isn't working

Comments

@flokraft85
Copy link

Describe the bug
When clicking or the first time on the variant icon for flags, comments and ACMG, the links leads to the top of the variant details page and not to the specific position for flags, comments or ACMG. After going back from the variant detail page to the variant filter page an clicking agian on one of these icons, the links leads now to the specific postion. This is the case for every variant. So the first time clicking on an icon the linkt leads to the top of the variant details page. Going back to the variant filter page and clicking again leads to the specific postion on the variant details page.

To Reproduce
Steps to reproduce the behavior:

  1. Start on the variant filter page
  2. Click on one of the flags, comments or ACMG icons
  3. Using the back button, to go back to the variant filter page
  4. Click agian on one of the flags, comments or ACMG icons

Expected behavior
Clicking on the icons should lead to the specified postion on the variant details page.

Desktop (please complete the following information):

  • OS: Win10
  • Browser Firefox
  • Version 121
    le an clicking agian on one of these icons, the links leads now to the specific postion. This is the case for every variant. So the first time clicking on an icon the linkt leads to the top of the variant details page. Going back to the variant table and clicking again leads to the specific postion on the variant details page.

To Reproduce
Steps to reproduce the behavior:

  1. Start on the variant filter page
  2. Click on one of the flags, comments or ACMG icons
  3. Using the back button, to go back to the variant filter page
  4. Click agian on one of the flags, comments or ACMG icons

Expected behavior
Clicking on the icons should lead to the specified postion on the variant details page.

Desktop (please complete the following information):

  • OS: Win10
  • Browser Firefox
  • Version 121
@flokraft85 flokraft85 added the bug Something isn't working label Jan 31, 2024
@holtgrewe holtgrewe moved this from To triage to Backlog in Release Planning Feb 1, 2024
@xiamaz xiamaz moved this from Backlog to Ready in Release Planning Mar 21, 2024
@stolpeo stolpeo moved this from Ready to In progress in Release Planning Mar 21, 2024
@stolpeo
Copy link
Contributor

stolpeo commented Mar 21, 2024

This changed with the integration of REEV. I will re-evaluate the behavior.

@stolpeo
Copy link
Contributor

stolpeo commented Mar 22, 2024

RCA

The issue persists with REEV, but the root cause is different. The router library is used to address the section to jump to, but the page builds too slowly as it is dynamically loaded so that the jump can't take place.

@stolpeo
Copy link
Contributor

stolpeo commented Mar 22, 2024

Solution proposal

It is clear that we can't make router wait for the page to be loaded.

  1. Either there is a possibility to get the jump position from the router call and execute it again after the page has fully loaded.
  2. Or we store the jump position in the store when details are clicked and jump to it manually when the page finished loading.

@holtgrewe
Copy link
Collaborator

@stolpeo the router does not jump itself. Rather it passes a prop into the top level Vue view. That is, in the view you can wait/react on that prop and all stores to finish loading.

Be careful with watching store state, this often triggers loops. Rather set a periodic timer that runs every 100ms and jumps once after everything has loaded and then deactivates itself by some means.

@holtgrewe
Copy link
Collaborator

@stolpeo actually watching store state for jumping should be ok as we don't write. Calling into store on watching store state can be dangerous and should be done after careful consideration. E.g. done -> trigger load -> done ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants