inf112-v20 / crawling-crow

🤖 The winning team!
0 stars 0 forks source link

Refactored UI Elements #213

Closed timharek closed 4 years ago

timharek commented 4 years ago

162 #209

thomashusebo commented 4 years ago

Bør gjerne løs bug #215 før vi merger flere branches? Eventuelt reverte develop til en tidligere stadie først?

timharek commented 4 years ago

Bør gjerne løs bug #215 før vi merger flere branches? Eventuelt reverte develop til en tidligere stadie først?

Fikset!

thomashusebo commented 4 years ago

Ettersom det er merge conflicts, så foreslår at develop merges inn i denne branchen, slik at vi kan se at mergen fungerer slik den er tenkt. Kan gi en review når dette er gjort.

buzzdead commented 4 years ago

Når man trykker på et kort "av og på", flere ganger, så flytter indeksen seg mot venstre for hvert klikk. Her på bildet har jeg valgt kort 1, 2, og 3 flere ganger:

image

Dette kommer nok av at den henter posisjonen til den tidligere selectedorderLabel og trekker fra tempselectedorder greia i linje 133 til 136 i programcradsview, hvis noen ønsker å fikse det fort så er det sikkert nok å gjøre den posisjoneringen en gang et sted og ikke trekke fra i disse linjene.

timharek commented 4 years ago

Når man trykker på et kort "av og på", flere ganger, så flytter indeksen seg mot venstre for hvert klikk. Her på bildet har jeg valgt kort 1, 2, og 3 flere ganger: image

Dette kommer nok av at den henter posisjonen til den tidligere selectedorderLabel og trekker fra tempselectedorder greia i linje 133 til 136 i programcradsview, hvis noen ønsker å fikse det fort så er det sikkert nok å gjøre den posisjoneringen en gang et sted og ikke trekke fra i disse linjene.

Fikset!

thomashusebo commented 4 years ago

Godt jobba!