Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Posizionamento dei label #79

Closed
valeriosantarelli opened this issue Mar 21, 2020 · 7 comments
Closed

Posizionamento dei label #79

valeriosantarelli opened this issue Mar 21, 2020 · 7 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@valeriosantarelli
Copy link
Contributor

Mi sono reso conto che i label sui nodi (di qualsiasi tipo, IRI, prefixedIRI, o proprio label), se vengono spostati, alla successiva apertura di Eddy poi vengono riportati alla posizione originale.

Ad esempio, guarda i due screenshot sotto (stessa ontologia, subito prima e subito dopo la chiusura e riapertura di Eddy): i label di diversi ruoli sono stati riportati al centro del nodo, così come la prefixedIRI del concetto woc:Manufacturer.

Schermata 2020-03-21 alle 19 28 58

Schermata 2020-03-21 alle 19 29 40

@valeriosantarelli valeriosantarelli added the bug Something isn't working label Mar 21, 2020
@valeriosantarelli valeriosantarelli added this to the Eddy v2 milestone Mar 21, 2020
NonnonePalmiro pushed a commit that referenced this issue Mar 22, 2020
@valeriosantarelli
Copy link
Contributor Author

Ho riprovato, e in effetti i label non vengono riposizionati più al centro del nodo.
Però adesso si ha come un effetto contrario: vengono spostati rispetto al piazzamento deciso dall'utente.

Prima di chiudere:
Schermata 2020-03-23 alle 11 17 32

Dopo aver riaperto:
Schermata 2020-03-23 alle 11 18 06

@valeriosantarelli
Copy link
Contributor Author

@mnamici @NonnonePalmiro Ragazzi, io e Giacomo ci siamo accorti che il problema del posizionamento del label è presente già nella versione 1.2 di Eddy.

Ho fatto una prova stupidissima, inserendo un concetto e un attributo, e spostando la label, e alla successiva riapertura le etichette sono spostate.

A questo punto è urgente trovare una soluzione più rapidamente possibile e fare un nuovo rilascio, perchè stiamo già usando Eddy per il progetto con Inail. Valutate voi, non appena il problema è risolto, se è il caso di fare direttamente un rilascio della 3.0, oppure una release 1.3.

@mnamici
Copy link
Collaborator

mnamici commented May 1, 2020

@mnamici @NonnonePalmiro Ragazzi, io e Giacomo ci siamo accorti che il problema del posizionamento del label è presente già nella versione 1.2 di Eddy.

Ciao ragazzi, sto provando a riprodurre il problema nella 1.2 ma non ci riesco, riepilogo gli step che ho provato:

  1. Creato nuovo progetto e creato un diagramma.
  2. Inserito un nuovo concetto
  3. Inserito un nuovo attributo
  4. Spostato entrambe le label fuori dalla posizione di default.
  5. Chiuso e riaperto il progetto più volte ma la posizione rimane corretta.

Puoi elencarmi esattamente gli step che avete seguito per riprodurre la issue?

@NonnonePalmiro
Copy link
Contributor

Io ho appena scaricato la 1.2 da drive e anche a me funziona bene

@valeriosantarelli
Copy link
Contributor Author

Ho scaricato anche io ex-novo la 1.2, e fatto esattamente gli stessi passaggi che ha elencato Manuel, e in effetti rimane tutto a posto.
Prima avevo fatto gli stessi passaggi, e il comportamento era diverso.

Non ho idea del perchè questo fenomeno si stia verificando nell'ontologia che vedete in allegato.
L'unica cosa che ha di particolare (da quello che riesco a vedere) è che in questa ontologia ci sono delle occorrenze di label dei nodi concetto che sono stati spostati fuori dal rettangolo.
INAIL.zip

@giacomoronconiobda potresti fare una prova scaricandoti ex-novo la versione 1.2, sistemando un pò dei label spostati, e vedendo se vengono rispostati alle successive aperture?

@giacomoronconiobda
Copy link

giacomoronconiobda commented May 2, 2020 via email

mnamici added a commit that referenced this issue May 6, 2020
The introduction of customizable font sizes for nodes in 9eae21d
introduced also a new feature that allowed for labels to preserve their
original position upon a change in the label's bounding box.
This is done to preserve the text alignment with the graphical node.
However, the previous implementation was causing labels to move slightly
from their default position due to a misused call to QGraphicsTextItem::adjustSize().

This commits fixes the behaviour introduced in 9eae21d by making
sure that the position is preserved upon font changes for labels.

Fixes issues described in #79 for v1.2
@valeriosantarelli valeriosantarelli modified the milestones: Eddy v1.3, Eddy v3 May 8, 2020
@valeriosantarelli
Copy link
Contributor Author

La issue è risolta a partire dal build del 26/05/2020. Chiudo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants