Semantic-Org/Semantic-UI-React

Popup: Incorrect position of arrow

Open

#3771 aperta il 2 set 2019

Vedi su GitHub
 (7 commenti) (7 reazioni) (0 assegnatari)JavaScript (4065 fork)batch import
bughelp wantedneeds engineering

Metriche repository

Star
 (13.136 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il bug si trova nella logica di posizionamento del componente Popup. Il problema fornisce un test case su CodeSandbox (https://codesandbox.io/s/semantic ui react o7mfd). I manutentori devono indagare sul motivo per cui la proprietà position non viene ricalcolata in base alla posizione effettiva dell'elemento trigger, come avveniva nella versione 0.86.0. I file probabilmente correlati sono `src/modules/Popup` e l'utilità di posizionamento.
Tech stack
javascriptreactcss
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
React
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60