Semantic-Org/Semantic-UI-React

Popup: Incorrect position of arrow

Open

#3.771 geöffnet am 2. Sept. 2019

Auf GitHub ansehen
 (7 Kommentare) (7 Reaktionen) (0 zugewiesene Personen)JavaScript (4.065 Forks)batch import
bughelp wantedneeds engineering

Repository-Metriken

Stars
 (13.136 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Bug Report

Steps

  1. Create an element with a popup
  2. Move that element to the right side of the page

Expected Result

I don't know an exact position of a trigger element on a page in advance but I expect popup and popup arrow will be displayed correctly everywhere as it was in 0.86.0. old

Actual Result

Popup arrow position is incorrect. The package doesn't calculate a popup position based on an actual popup trigger position. It passes down a position from props to popper and applies css styles based on that instead of calculating a correct value. In the previous version a position was changed to 'top right' but in current version it's 'top left' from defaultProps always. new

Version

0.88.0

Testcase

https://codesandbox.io/s/semantic-ui-react-o7mfd

Contributor Guide

Research-Richtung
Der Bug liegt in der Positionierungslogik der Popup Komponente. Das Problem stellt einen CodeSandbox Testfall bereit (https://codesandbox.io/s/semantic ui react o7mfd). Die Maintainer müssen untersuchen, warum die Position Eigenschaft nicht basierend auf der tatsächlichen Position des Trigger Elements neu berechnet wird, wie es in Version 0.86.0 der Fall war. Wahrscheinlich betroffene Dateien sind `src/modules/Popup` und das Positionierungs Utility.
Tech Stack
javascriptreactcss
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
React
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60