Zoaklen / lolbuildcalculator

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

[REQ] Unificar a função de heurística com a função de condição #4

Closed Zoaklen closed 2 years ago

Zoaklen commented 2 years ago

Problema

Na aba de heurística, o código da função da heurística e a condição da combinação dividem o espaço horizontal da tela igualmente, o problema é que métodos grandes como o DamageTester.applyDamage() costumam ultrapassar o tamanho do painel em monitores menores.

Possível solução

Seria interessante unificar os dois painéis em um só, dessa forma haverá 100% do espaço horizontal para o código, o que soluciona o problema da falta de espaço horizontal.

Informações adicionais

@VitorGSF acredito que você tenha experiência o suficiente com o Swing do Java pra executar essas mudanças. image

VitorGSF commented 2 years ago

Soluções pesquisadas

Pesquisei sobre formatações possíveis de fazer em relação ao encontrado no problema, penso que a melhor formatação para sanar tal problema possa ser a utilização de ambos os painéis utilizarem 100% do espaço horizontal e utilizar o espaço vertical para ocupar um painel sob o outro, como encontrado neste link: https://pt.stackoverflow.com/questions/237966/posicionar-jpanels-na-vertical. Acredito que, assim como foi pedido e como me foi possível pensar em uma solução, possa ocorrer a unificação dos trechos de código respectivos, pois baseiam-se de uma mesma classe e importações, o que as torna dois métodos que possam coexistir no mesmo espaço. Caso estiver equivocado, me corrija.

Outras soluções possíveis

Também pesquisei soluções para manter a formatação e observando o apresentado no link (https://docs.oracle.com/javase/tutorial/uiswing/layout/box.html) penso que seja possível fazer quebra de linha com o conteúdo dentro do painel.

VitorGSF commented 2 years ago

Adicionado o commit 59d8135 para solução do problema.