DSpace / dspace-angular

DSpace User Interface built on Angular.io
https://wiki.lyrasis.org/display/DSDOC8x/
BSD 3-Clause "New" or "Revised" License
123 stars 394 forks source link

DSpace 7: Page jumps during submission #2781

Open f-rapp opened 5 months ago

f-rapp commented 5 months ago

Describe the bug During submission, the page sometimes "jumps" and users have to navigate back to where they were before the jump (scroll up/down). We noticed the page jumps occur when DSpace caches the submission (the caching is visible at the bottom of the page).

To Reproduce Steps to reproduce the behavior:

We have heard from other institutions that face the same problem and do not have a solution for this issue so far.

Expected behavior When DSpace caches a submission, the page does not "jump".

Related work Link to any related tickets or PRs here.

alanorth commented 5 months ago

@f-rapp I've noticed this too. Do you have type binding enabled in your submission form by any chance?

Some other factors could be: when you have more than one submission step. We have three, and DSpace attempts to auto save when the users moves between them, and the page jumps to the bottom.

f-rapp commented 5 months ago

@alanorth Yes, we use type binding in our submission form. We also have more than one submission step.

alanorth commented 5 months ago

Oh yeah, this jumpiness is the minor issue. The major issue is that when it jumps, the previously selected values are cleared. It is very reproducible for me on DSpace 7.6.1 in Firefox:

Screencast from 2024-01-30 11-38-39.webm

You can see me skip to the third "Describe" step and try to select values in some drop downs. When I move back to the second step the autosave kicks in and clears the field, jumping sometimes and others not. Is this the same experience you have @f-rapp?

I think this is a high-priority issue because it makes submission nearly impossible unless you are VERY persistent and meticulous about the order you complete the submission form.

f-rapp commented 5 months ago

I have tested this again and while we have the jumps, I didn't find any cleared values.

alanorth commented 5 months ago

I'm testing again and the "cleared values" bug is actually a side-effect of recently merged #2733. It is not related to the jumping submission form in this issue.

f-rapp commented 3 months ago

Hi @alanorth, does the patch above also solve the jumping issue?

alanorth commented 3 months ago

Hi @f-rapp. No that patch does not fix the jumping...