Gabboxl / ClassevivaPCTO

An UWP app to consult Classeviva's website
https://www.microsoft.com/store/productId/9PNST3M88D1S
GNU General Public License v3.0
8 stars 1 forks source link

Code quality & string improvements, UI Updates, New palettes, Various fixes + reformat XAML #249

Closed alessandrocaseti closed 6 months ago

alessandrocaseti commented 6 months ago

closes: #241 closes: #250 closes: #246 closes: #240 closes: #230 implementato uno stile predefinito per le proprietà path degli hyperlink implementato uno stile predefinito per i popup in agenda implementato uno stile predefinito per i border della combobox palette aggiunto button chiudi flyout (notice) commentati visualstate inutilizzati aggiunte 2 nuove palette: Pastello e Vintage aggiunto colore giallo nella combobox palette migliorata la sintassi del dialog untranslatedlanguage implementate PRI resource delle lingue nuove #229 ottimizzate le strings ridondanti tanti, ma tanti fix e miglioramenti all'UI aggiunto placeholder nessun allegato (noticedialog) reformat XAML per views e dialogs fix: la NoticeListView si aggiornava anche quando veniva aperta una comunicazione già letta

@Gabboxl 2 cose: -- VS ha segnato come errore la proprietà ShouldConstrainToRootBounds dei pupup in agenda in quanto non supportata sulla versione minima dichiarata. L'ho aggiornata a 10.0.18362.0 per risolvere, sempre sotto consiglio di vs -- Le lingue polacco e albanese non triggerano il dialog e da debug risulta esserci un exception dovuta alle string inesistenti, che facciamo le togliamo?

Gabboxl commented 6 months ago

se riesci magari la prossima volta separiamo le modifiche di formattazione e le nuove implementazioni in due pull request separate, così riesco a capire al volo cosa hai fatto di nuovo @alessandrocaseti

Gabboxl commented 6 months ago

inoltre se in ambiti diversi, come più issue che riguardano parti diverse dell'app, cerca di separare il codice in pull request dedicate

alessandrocaseti commented 6 months ago

@Gabboxl fatto tutto. Il messaggio di avviso che mi dava è

XLS1105 La proprietà 'ShouldConstrainToRootBounds' nel tipo 'Popup' non è disponibile nella versione minima della piattaforma di destinazione (10.0.17763.0) supportata da questo progetto. ClassevivaPCTO C:\Users\alex0\Documents\GitHub\ClassevivaPCTO\ClassevivaPCTO\Views\AgendaPage.xaml 28

lo ignoriamo?

poi, se riesci controlla le lingue come ti ho scritto alla fine della descrizione della pr

Gabboxl commented 6 months ago

@Gabboxl fatto tutto. Il messaggio di avviso che mi dava è

XLS1105 La proprietà 'ShouldConstrainToRootBounds' nel tipo 'Popup' non è disponibile nella versione minima della piattaforma di destinazione (10.0.17763.0) supportata da questo progetto. ClassevivaPCTO C:\Users\alex0\Documents\GitHub\ClassevivaPCTO\ClassevivaPCTO\Views\AgendaPage.xaml 28

lo ignoriamo?

poi, se riesci controlla le lingue come ti ho scritto alla fine della descrizione della pr

effettua un revert delle modifiche che hai fatto ai popup, lasciamo quella proprietà sull'oggetto come prima

alessandrocaseti commented 6 months ago

@Gabboxl fatto, ho anche corretto il margin del calendario assenze

alessandrocaseti commented 6 months ago

direi che ci siamo, per quella cosa delle strings non saprei come fare

Gabboxl commented 6 months ago

direi che ci siamo, per quella cosa delle strings non saprei come fare

fai un revert, faccio io più tardi

alessandrocaseti commented 6 months ago

direi che ci siamo, per quella cosa delle strings non saprei come fare

fai un revert, faccio io più tardi

eh ma dopo come fai con le path e le data? io ho trovato quel sistema che non credo dia problemi, casomai prova a farlo meglio come dicevi

Gabboxl commented 6 months ago

direi che ci siamo, per quella cosa delle strings non saprei come fare

fai un revert, faccio io più tardi

eh ma dopo come fai con le path e le data? io ho trovato quel sistema che non credo dia problemi, casomai prova a farlo meglio come dicevi

si lascia le strings e togli questa parte di codice, sarà rotto temporaneamente

alessandrocaseti commented 6 months ago

ok

alessandrocaseti commented 6 months ago

pronta