boapps / Szivacs-Naplo

Egy multiplatformos kliensalkalmazás az e-napló rendszerhez.
BSD 2-Clause "Simplified" License
59 stars 10 forks source link

Főoldali összefoglaló kártyák, 200%-os jegyek kiemelése, egyéb javítások #97

Closed RedyAu closed 4 years ago

RedyAu commented 4 years ago

globals.colorF1, F2, F3, F4, F5 automatikusan frissül a színek átállításakor számítás alapján fehérre/feketére, így mindig látható a jegyen a felirat.

A kártyákra került egy körvonal, hogy jobban összetartozzanak az alsó, nem színes sávjukkal. A súlyozott jegyek körvonala vastagabb.

A backend meg tudja különböztetni, és helyesen kezeli az 1, 3. negyedéves jegyeket. Új backend funkció: bool Evaluation.isSummaryEvaluation Igaz, ha az adott jegy negyedéves, félévi, vagy év végi. Az ilyen jegyek többé nem kerülnek ki a főoldali kártyák közé, csak a közös dobozba.

Főoldalon "Info" gomb létrehozva, hogy el lehessen érni a ""felturbózott"" ester egget.

És még további apróságok, újrarendezések, kommentelések.

cameralis commented 4 years ago

Szerintem ha a körvonal megegyezne a kártya színével az szebb lenne, de amúgy tetszik.

RedyAu commented 4 years ago

Szerintem ha a körvonal megegyezne a kártya színével az szebb lenne, de amúgy tetszik.

A jegykártyára vagy az összefoglaló kártyára gondolsz?

cameralis commented 4 years ago

Szerintem mindkettőre. És valahogy a 200%-os jegyek körül lévő piros körvonal egy kicsit fúrcsán néz ki.

cameralis commented 4 years ago

Esetleg ha csak vastagabbak lennének a jegyek?

RedyAu commented 4 years ago

Szerintem az összefoglalónak szép kontúrt ad, hogy kicsit más színű, viszont a jegyeknél egyetértek, azt megcsinálom. A piros körvonal tényleg nem fullos, kéne helynek lenni a karika és a pötty között... Megpróbálkozok vele :D

RedyAu commented 4 years ago

Sajnos ebben a betűtípusban nem túl nagy a különbség, és úgy nem lenne jelezve az esetleges 300, 400%-os jegyek közötti különbség (ami amúgy nem tragédia)

RedyAu commented 4 years ago

image image image image

RedyAu commented 4 years ago

Nekem így egész tetszik :P

cameralis commented 4 years ago

Nekem is, de hogyha a körvonalak színét változtatod akkor szerintem a vastagságát már nem kell + lehetne egy kicsit kevesebb a kontraszt a körvonalaknál és le lehetne kerekíteni a kártyát is (nem csak a körvonvalat). image

RedyAu commented 4 years ago

A conflictocat megoldottam, de még ne mergeljünk! Írok még bele pár dolgot.

RedyAu commented 4 years ago

Nekem is, de hogyha a körvonalak színét változtatod akkor szerintem a vastagságát már nem kell + lehetne egy kicsit kevesebb a kontraszt a körvonalaknál és le lehetne kerekíteni a kártyát is (nem csak a körvonvalat).

Nem találtam sajnos módját az alsó doboz lekerekítésének, a hátteret mindenképp felé rajzolja a körvonalnak. Feel free to fix it everybody. A vastagságra is szükség van, mert különben alig-alig van különbség, viszont nem akarom hogy alapból minden vastag legyen, az is csúnya.

cameralis commented 4 years ago

Az info screen-nél tíltsd le a side menu-t mert különben ez lesz

0-02-0a-5ea7ad58bbb01341197cb832a64e41ce54ad2eeae16f5f3e205770fec3147041_1f9fd78ae8a01f

cameralis commented 4 years ago

Btw. tetszik a hiperhajtómű ;)

boapps commented 4 years ago

Ügyes, de bocsi, amíg az benne van, addig nem fogadom el a PR-et. :pensive:

RedyAu commented 4 years ago

Ok, kiveszem, valamelyik forkba belekerülhet esetleg ;) Én nem fogok saját projektet csinálni belőle. Az info screent is kiveszem, így nem kell.

RedyAu commented 4 years ago

image

RedyAu commented 4 years ago

cameralis commented 4 years ago

És ez milyen?

0-02-05-f2451a48a25daae2675c102ebe445ce03f48c2b38ca6f8e05461d69215c6e5d3_1f9fd78a98b1c8

Nem 100%-os

0-02-0a-2e9f6d68cc7ca0240a62d792ca45879b667990732e0e573d596e52f59dff9a7b_1f9fd78a8389e1

RedyAu commented 4 years ago

Verrry nice indeed!

boapps commented 4 years ago

nekem ez az új design nagyon tetszik

RedyAu commented 4 years ago

Na kiveszem gyorsan a backdoort hogy lehessen az eddigieket mergelni, és ez is belejöhessen, 5 perc

cameralis commented 4 years ago

Nem működik a "tárgyanként" tab

RedyAu commented 4 years ago

Nem ám, most kezdtem el csinálni

cameralis commented 4 years ago

Ó rendben

RedyAu commented 4 years ago

Kicsit logikusabbra gondoltam csinálni, összevonom a jegyek és statisztikák oldalt, a jegyek tárgyankénti rendezése gyakorlatilag ugyanaz, mint a statisztikák oldal. Az átlagos, összesítő jegyes popup is az eddigi statisztika oldal alsó sávjáról lesz elérhető, csak külön oldalon, szépen megcsinálva, negyedévekkel

RedyAu commented 4 years ago

Te tervezed felküldeni a lekerekítést?

cameralis commented 4 years ago

Igen, egy pillanat

cameralis commented 4 years ago

Ha megpróbálom tárgy szerint sorbarendezni a jegyeket

0-02-05-cca2312b973d4138b8a60e4557da86fa333840519ef43aadad92559c8538a009_1f9fd789cf3599

RedyAu commented 4 years ago

Ezt mikor adta ki? Ééés légyszi commitold valahova a lekerekítést :)

cameralis commented 4 years ago

A lekerekítésnek csináltam egy PR-t: #99

Error log

E/flutter (26372): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: NoSuchMethodError: The getter 'length' was called on null.
E/flutter (26372): Receiver: null
E/flutter (26372): Tried calling: length
E/flutter (26372): #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
E/flutter (26372): #1      _StringBase.compareTo (dart:core-patch/string_patch.dart:290:29)
E/flutter (26372): #2      EvaluationsScreenState.refreshSort.<anonymous closure>.<anonymous closure> (package:e_szivacs/screens/evaluationsScreen.dart:175:30)

screens/evaluationsScreen.dart:175

image

cameralis commented 4 years ago

Mondjuk a tárgyak szerinti sorbarendezés a playes verzióban sem működik jól,de nem dob hibát

boapps commented 4 years ago

@UnknownPlayer78 mi nem jó benne? (nekem megy)

cameralis commented 4 years ago

Egy tárgy többször is van

boapps commented 4 years ago

hmm, nincs véletlen magatartás-szorgalom jegyed? Annál lehet, hogy az apiból lekért Subject null.

RedyAu commented 4 years ago

Én is már az egészet telespameltem ??""-el pont emiatt :D

A lekerekítésnek csináltam egy PR-t: #99

Sry, my bad

RedyAu commented 4 years ago

Nekem van magatartás-szorgalom, de a tárgy szerinti rendezés működik...

cameralis commented 4 years ago
a.TypeName = Évközi jegy/értékelés
b.TypeName = Évközi jegy/értékelés

Error log

E/flutter (13056): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: NoSuchMethodError: The getter 'length' was called on null.
E/flutter (13056): Receiver: null
E/flutter (13056): Tried calling: length
RedyAu commented 4 years ago

Ez tárgy szerinti rendezésnél ugye?

cameralis commented 4 years ago

igen

RedyAu commented 4 years ago

Az nem baj, már szétbarmoltam a tárgy szerinti rendezést, mert tárgy szerint az eddigi statisztika menüben lehet nézni.

cameralis commented 4 years ago

Akkor jó

RedyAu commented 4 years ago

Valszeg ezért volt rossz :D