Z3r0byte / Magistify

All your favorite Magister tools in one app
https://magistify.nl
Apache License 2.0
2 stars 2 forks source link

Roosterwijzigingen op dashboard #45

Closed ghost closed 7 years ago

ghost commented 7 years ago

Misschien is het een goed idee om de eerste drie (eventueel kun je dit getal instelbaar maken) aankomende roosterwijzigingen op het dashboard te tonen

ghost commented 7 years ago

De roosterwijziging-meldingen werken! Dat is erg goed nieuws. Nu kan ik morgenochtend testen of de "melding als volgende les gewijzigd is" functie werkt.

Z3r0byte commented 7 years ago

Ik zie je comment van 5 dagen geleden nu pas, sorry voor dat 😅. Het is inderdaad wel een goed idee om een paar roosterwijzigingen te laten zien. En goed nieuws om te horen dat tot nu toe alles werkt! Als het goed is zou je de melding voor de komende les 25 minuten van te voren moeten krijgen.

ghost commented 7 years ago

@Z3r0byte Hmm dan zou ik hem inmiddels al gehad moeten hebben. Misschien komt het omdat de les uitgevallen is dus hij "bestaat niet meer". Misschien is dit onmogelijk, maar is het mogelijk om ook de oude situatie te tonen in het roosterwijzigingen scherm?

Sent from my Samsung kminilte using FastHub

Z3r0byte commented 7 years ago

Dit zou moeilijk worden, omdat in de agenda ook al de bijgewerkte afspraken staan, je kunt dus niet gaan vergelijken.

ghost commented 7 years ago

@Z3r0byte Ja maar ik dacht dat de Magister API dit misschien ook meestuurde. De melding voor de volgende les werkt trouwens nog niet

Sent from my Samsung kminilte using FastHub

Z3r0byte commented 7 years ago

Oké, bedankt voor het doorgeven, ik zal kijken of ik vandaag tijd heb om het op te lossen.

Z3r0byte commented 7 years ago

De melding werkt in de nieuwste release als het goed is

ghost commented 7 years ago

@Z3r0byte Ja hij werkt!

Sent from my Samsung Galaxy S5 using FastHub

Z3r0byte commented 7 years ago

Mooi zo :)

Z3r0byte commented 7 years ago

Ik ben hiermee aan de slag gegaan, maar er zijn nog een paar bugs die moeten worden opgelost ;)

Z3r0byte commented 7 years ago

@StanNL UPDATE: ik heb de bugs als het goed is opgelost (moet het nog wel even testen op emulators), dus de update komt binnenkort (ook al heb je daar waarschijnlijk niet meer veel aan 😆 )

ghost commented 7 years ago

Het werkt helemaal. Hij toont de eerste drie roosterwijzigingen voor de komende schooldag. Het is echter zo dat de roosterwijzigingen activity niet opent als je op een individuele roosterwijziging tikt. Ook ontbreekt de schaduw van de card aan de onderkant. Verder werkt het heel goed!

Z3r0byte commented 7 years ago

@StanNL Dat van die schaduw klopt, en ik heb het ook geprobeerd op te lossen, maar na een uur proberen heb ik het opgegeven 😂. Ik zal nog een klikactie toevoegen aan de individuele roosterwijzigingen.

Z3r0byte commented 7 years ago

Ik zie trouwens dat de landscape layout nog niet klopt, ik zal die binnenkort verbeteren

ghost commented 7 years ago

Ik denk dat ik al zie waar het "schaduwprobleem" vandaan komt. Aan de onderkant van de activity zit een grijze streep. Om het in CSS uit de drukken (omdat ik niks van Java afweet): De body heeft een te grote padding, of juist een margin. Als je die waardes tweakt, of als je die card een grotere margin-bottom geeft, denk ik dat je het probleem kunt oplossen.

Z3r0byte commented 7 years ago

Het heeft inderdaad iets met de margin te maken denk ik. Ik heb ook al met de margin en padding zitten spelen, maar zonder succes :(

Z3r0byte commented 7 years ago

De landscape view is zonder roosterwijzigingen, omdat ik het niet voor elkaar kreeg om die fatsoenlijk zonder overlap onder de cijfers en lessen widget te plaatsen