inf112-v20 / crawling-crow

🤖 The winning team!
0 stars 0 forks source link

[BUG] [UI] Some UI elements (when resizing) are not reacting to input when clicked on #166

Closed timharek closed 4 years ago

timharek commented 4 years ago

Noen UI elementer reagerer ikke riktig når man klikker på dem, man er nødt til å klikke litt til siden av dem.

buzzdead commented 4 years ago

Teste på forskjellige oppløsninger, gjerne på TV osv før man til slutt closer denne? Stort sett bør det være ok å teste på PC så er vel ikke så utrolig nøye.

timharek commented 4 years ago

Har testet på min store PC skjerm med 2560x2440 og satt MacBooken til 2048x1280, det er en god del høyere enn 1920x1080 og det funkerte kjempebra!

Men når jeg testet å nedskalere MacBooken til 1280x720, så ser UI'en veldig stor ut. Problemet på macOS er at man ikke får lov å ha større vindu enn hva oppløsningen tilsier, og jeg mener det ikke er en begrensning på Windows.

Skjermbilde 2020-04-30 kl  14 10 32

Så jeg lurer på om dette er noe vi burde spør Fromsa om mtp om vi skal kunne støtte Mac med lav oppløsning, for det er kun her jeg tror det blir et problem.

timharek commented 4 years ago

Tror jeg har en fix på det overnevnte problemet, er ikke hjemme akkurat nå, men skal prøve å få testa fixen senere i dag

timharek commented 4 years ago

Tror jeg har en fix på det overnevnte problemet, er ikke hjemme akkurat nå, men skal prøve å få testa fixen senere i dag

Nei, det funka ikke alikevel, resten av UI'en følgte ikke med, så det ble ikke resizet som jeg hadde ønsket :(

buzzdead commented 4 years ago

Om problemet bare er at UIen ser stor ut så er ikke det noe problem.

timharek commented 4 years ago

Om problemet bare er at UIen ser stor ut så er ikke det noe problem.

Har du en fiks på det da ev.? For fiksen er relativt enkel, linje 79 i GameView

camera.setToOrtho(false, SettingsUtil.WINDOW_WIDTH, SettingsUtil.WINDOW_HEIGHT);
buzzdead commented 4 years ago

Denne vi implementerte var det ikke? Jeg trodde du mente at UI elementene ble store på mac med lav oppløsning -etter- denne fiksen? Kunne eventuelt hatt en knapp "If you are running mac with low res click this button"

timharek commented 4 years ago

Dette er en endring, egentlig så står det

camera.setToOrtho(false, Gdx.graphics.getWidth(), Gdx.graphics.getHeight());

Det vi endra på tidligere var FitViewport(). Skjønner det ser veldig likt ut 😅

buzzdead commented 4 years ago

Hmmm, så lenge det ikke forrandrer på noe annet så er det vel bare å skifte den da hvis den fikser mac med lav res også?

timharek commented 4 years ago

Den fikser mac med lav res, men lager et nytt problem siden UI'en skalerer ikke på samme nivå...

Skjermbilde 2020-04-30 kl  18 07 03
buzzdead commented 4 years ago

Vurdere å lage en mac med lav res option da i menyen.

timharek commented 4 years ago

Jupp, med mindre Fromsa sier vi ikke trenger å tenke på det.

timharek commented 4 years ago

Kom over en tilfeldig ting; Hvis man setter cfg.useHDPI = true i Main, så forsvinner problemet på moderne maskiner som skalerer pga høy oppløsning, men det skaper nye problemer igjen, se bilde.

Skjermbilde 2020-05-08 kl  01 08 15

Var helt tilfeldig at jeg kom over dette, har ikke planer om å fikse dette.