Semantic-Org/Semantic-UI-React

Dropdown scroll position going top when adding options

Open

#3.721 geöffnet am 26. Juli 2019

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (13.136 Stars) (4.065 Forks)batch import
help wantedneeds engineering

Beschreibung

Hello,

I did search in the 30+ issues opened but did not see anything similar. Here is the bug I've encountered :

Bug Report

When modifying options of a Dropdown while you are scrolling inside of it, the scroll position going full top.

Expected Result

Scroll staying on actual position

Actual Result

Scroll reseting to top

Version

SUIR@0.87.3

Testcase

In this Testcase, I simulated addition of elements when clicking on a button. FYI, in my reality case, I am doing an infinite scroll, which is adding elements when needed.

https://codesandbox.io/s/semantic-ui-react-example-1xvv7?fontsize=14

Contributor Guide

Tech Stack
javascriptreact
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
ReactSemantic UI React basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
The issue reports that the Dropdown component's scroll position resets to top when its options are modified while scrolling. The testcase in the issue shows adding options via a button. The likely cause is that the component re renders and resets the scroll position. Investigate the Dropdown component source code, typically in `src/modules/Dropdown/Dropdown.js`. Look for how the scroll position is managed, possibly via a ref or state. The fix may involve preserving the scroll position before re render and restoring it after. Check if there are any lifecycle methods or hooks that handle this. Since the issue is old and unassigned, no linked PRs or comments exist.