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

Multiple author affiliations and RORs - Entering affiliations jumps the screen around after each letter #10904

Open
Tribunal33 opened this issue Feb 7, 2025 · 3 comments
Assignees
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Milestone

Comments

@Tribunal33
Copy link

Describe the bug
While typing in a new Contributor affiliation the screen bounces around after each letter making it difficult to focus on and finish typing

To Reproduce
Steps to reproduce the behavior:

  1. Add a contributor by either going through submission workflow or adding to existing submission
  2. Make sure the field is postioned in the upper 1/s of the screen height
  3. Under Affiliations start entering "University of"
  4. After every additonal letter typed the screen bounces
  5. See video
    What application are you using?
    OJS 3.5

Additional information

Submissions._.Journal.of.Public.Knowledge.-.Google.Chrome.2025-02-06.15-27-57.mp4
@Tribunal33 Tribunal33 added the Bug:1:Low A bug that does not have a severe consequence or affects a small number of users. label Feb 7, 2025
@Tribunal33 Tribunal33 added this to the 3.5 Internal milestone Feb 7, 2025
@GaziYucel
Copy link
Contributor

@jardakotesovec could you have a look at this one. It seems that maybe the solution is in the Autosuggest component. Thanks.

@jardakotesovec jardakotesovec self-assigned this Feb 10, 2025
@jardakotesovec
Copy link
Contributor

@GaziYucel Yes will look into it.. but probably next week.

Main problem is that the dropdown thats showing/hiding often is changing height of the page which makes the jumping.

Two things come to mind that could improve that.

  1. Add delay before sending search request (debounce), thats something we should do anyway, so we are not hitting API too much.
  2. limit height of the dropdown and make it scrollable

GaziYucel added a commit to GaziYucel/ui-library that referenced this issue Feb 10, 2025
@GaziYucel
Copy link
Contributor

@jardakotesovec I was also thinking in the same direction. See this PR pkp/ui-library#520. Would this suffice for your second point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Projects
None yet
Development

No branches or pull requests

3 participants