obdasystems / eddy

A graphical editor for the specification and visualization of Graphol ontologies
GNU General Public License v3.0
56 stars 7 forks source link

Snake/camel case conversion in label generation #241

Closed MariaRosariaFraraccio closed 1 year ago

mnamici commented 1 year ago

Le modifiche che abbiamo elencato durante l'ultimo meeting sono complete qui? Se sì vorrei fare un refactor della componente grafica questa settimana e poi fare il merge.

MariaRosariaFraraccio commented 1 year ago

Sì, c'è tutto

mnamici commented 1 year ago

Ok grazie, allora procedo con il refactor dell'interfaccia e aggiungo qui.

mnamici commented 1 year ago

Fatto il rebase che prima era basato sulla vecchia versione della PR per il label wrap automatico.

mnamici commented 1 year ago

Ok iniziando a fare la review mi sono accorto che ci sono numerose cose che non vanno bene nell'interfaccia dell'ontology manager, e di conseguenza impatta sulle modifiche qui.

Ad esempio ho notato che le preferenze sulla label automatica dalla IRI sono impostate con un comando, il che significa che di fatto viene vista come una modifica del progetto di cui puoi fare undo!

Da un occhiata al seguente comportamento: bug

Se non bastasse che mi fa fare undo i setter si sono pure inventati di invertire i valori delle altre impostazioni che non ho toccato.

Questo ci dice che dobbiamo tenere un'attimo da conto la logica di conversione camel/snake case ma prima di applicarla dobbiamo sistemare l'interfaccia. Faccio una PR separata, poi una volta sistemato il manager applichiamo lì la logica di trasformazione snake e camel case.