Isso foi feito intencionalmente durante o desenvolvimento do recurso, com a ideia de seguir o exemplo do modal de favoritos do Firefox… A lógica é:
- Clique em “novo favorito”, clique fora do modal, o modal é fechado e o favorito é criado
- Clique em “novo favorito”, clique em cancelar, nenhum favorito é criado
- Clique em um favorito existente, um novo botão “Remover favorito” aparece, você pode usá-lo para removê-lo.
Isso não significa que não devêssemos alterá-lo se estiver confuso, mas tecnicamente não é um bug. Vou mover isso para Contribute > UX. A diferença entre um modal de página inteira (Discourse) e um menu flutuante (Firefox) pode ser significativa o suficiente para justificar a alteração do comportamento?