Zoaklen / lolbuildcalculator

League of Legends Item Build Calculator
0 stars 0 forks source link

[REQ] Adicionar função CTRL+Z #10

Closed AgenteVIIX closed 2 years ago

AgenteVIIX commented 2 years ago

Problema

Estava testando algumas heurísticas e percebi que não é possível utilizar o comando CTRL+Z dentro do editor de texto do programa, seria interessante adicionar essa funcionalidade pois é um padrão da maioria dos sistemas que trabalha com edição do texto.

Solução

Uma possível solução para o problema seria utilizar a classe UndoManager.

Informações adicionais

Neste link existe um tutorial detalhado de como implementar essa função: https://www.campcodes.com/tutorials/undo-text-using-ctrl-z-in-java/

@Zoaklen poderia implementar essa funcionalidade?

Zoaklen commented 2 years ago

Fiz um commit d16b00b que resolve o problema descrito.

O commit não soluciona o mesmo problema para a função de condição pois estou a espera da unificação das duas áreas de texto como descrito pela issue #4.

ClaudioACaramori commented 2 years ago

@Zoaklen Vi que a issue #4 foi resolvida e fechada, e só pra garantir, essa questão foi resolvida aqui também?

Zoaklen commented 2 years ago

@Zoaklen Vi que a issue #4 foi resolvida e fechada, e só pra garantir, essa questão foi resolvida aqui também?

Sim, foi resolvido assim como os conflitos de merge resultantes de mexer na mesma tela e nas mesmas funções da issue #4.