Semantic-Org/Semantic-UI-React

Popup: Incorrect position of arrow

Open

#3.771 aberto em 2 de set. de 2019

Ver no GitHub
 (7 comments) (7 reactions) (0 assignees)JavaScript (4.065 forks)batch import
bughelp wantedneeds engineering

Métricas do repositório

Stars
 (13.136 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
O bug está na lógica de posicionamento do componente Popup. O problema fornece um caso de teste no CodeSandbox (https://codesandbox.io/s/semantic ui react o7mfd). Os mantenedores precisam investigar por que a propriedade position não está sendo recalculada com base na posição real do elemento de gatilho, como era na versão 0.86.0. Os arquivos provavelmente relacionados são `src/modules/Popup` e o utilitário de posicionamento.
Pilha de tecnologia
javascriptreactcss
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
React
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60