oslandia / labelconnector Goto Github PK

It is probably an upstream issue in QGIS 3.4.9 - probably a regression

tested with a line layers, applying the style result in no label displayed until I modify the labeling properties. A signal is not cast somewhere, but not for all kinds of layers.

see the screencast:

The qml for this file :
line.zip

Break style currently leads to sometimes weird lines going backwards compared to old easycustomlabeling style:

That should probably be improved still I'd like to hear about user's preferences

Easy custom labeling always forced a '45°' heading for the line start, so that all connectors end to be aligned, making a more tidy view. Label connector computes a break at half length of the connector thus leading to different angles between connectors

Si on utilise labelconnector sur des lignes, le point de départ qui est fixé sur le centroid n'est pas correct.

Il faut utiliser une expression : line_interpolate_point($geometry, line_locate_point($geometry,centroid($geometry)))
comme point de départ.

We have this style:

we need this style like in former EasyCustomLabeling plugin

Les textes des boites de dialogue sont en anglais même si l'interface de QGIS est paramétré en français.
La traduction Anglais/français (fichier metadata.txt) de la phrase de présentation du plugin n'est pas tout à fait équivalente.
La version anglaise
'This plugin creates label connector (line from centroid to the label)'
n'indique pas clairement que les connecteur ne sont créés que pour les étiquettes déplacés manuellement.

Les traits de rappels peuvent être en-dessous de certaines entités.
Ce pb peut être résolu en activant le niveau de symbole dans les paramètres avancés.
Il serait bien que le plugin active automatiquement ce paramètre.

n_epci_zsup_r52.zip

Actuellement si on souhaite changer le style de connecteur il faut d'abord supprimer manuellement le générateur de géométrie de l'ancien connecteur dans le style. Ne serait-il pas possible de poser la question à l'utilisateur en cas de lancement une 2ème fois de labelconnector, puis le cas échéant de supprimer par programme l'ancien générateur, plutôt que d'afficher un message lui demandant de le faire manuellement ?

Easy Custom labelling laissait une marche entre le trait de rappel et l'étiquette. Cette petite marge serait à reprendre.

Si on active LabelConnector après une analyse graduées. Les icônes sont surchargées par le symbole du trait de rappel.

Ce qui n'est pas souhaitable.

Ce n'est pas le cas si on transforme le style gradué en ensemble de règles.

In this screen, I think we can get better namings:

This can be solved by adding symbol levels and setting the geometry generator's one higher than the others

Instead of this

We need a preference window with theses options:

• A combo box to choose default style

• a check box "Always show connector style window"

si on ajoute, après avoir déplacé des étiquettes, une règle qui n'affiche plus certaines étiquettes déplacées, le connecteur reste visible. Je pense que ce pb ne doit pas être facile à résoudre.
Au minimum avertir les utilisateurs dans la documentation de déplacer les étiquettes avant ajout de toute règles n'affichant pas toutes les étiquettes.

Dès activation de Label connector et après désignation de la clef primaire à utiliser, il remplace les étiquettes actuelles (par exemple NOM_COMM dans le fichier joint) par l'identifiant (dans l'exemple joint fid)
COMMUNES.zip

Si des étiquettes ont été définies par des règles puis que l'on utilise LabelConnector, les étiquettes reviennent à étiquettes simples et la définition des étiquettes est perdue.

Current logic applies a new geometry generator each time.
We could set tags into the geometry expression so that if a geometry generator is already there, with the tags, the plugin would only replace the expression

It is also a possibility to explain user where the expression comes from :

with this message.

One should be able to change the label connector style by calling the action several times

In this screen, I think we can get better namings:

Currently names are populated from filenames.
We need translatable aliases in full encoded string.

For instance straight should be Straight lines in english and Lignes simples in translation french

Having a simple lin (highway) shape, I thought to test the plugin.

So opened the shape (without saving it to a project...) and added normal labels.
Then tried to change them to 'connected labels' but did not see something happen.
Then removed labels and tried again to use the 'connected labels', but only see this now:

2019-06-21T08:56:57 CRITICAL LabelConnector already exists : If you want to add a new LabelConnector on the style, please remove the comment in the expression first.

BUT: I cannot find "the comment in the expression".
It could also have something to do with not having saved the project first?
Happy to sent you the (demo) shape.

When applying label connector to a a layer with no label, the labeling toolbar does not activate itself. I have to play with the settings and then this triggers the activation of the labeling toolbar.
I suspect we need to emit some signal in that case to activate the toolbar.

I propose this

pushed in 0b1de65

I'm not certain if this is necessary an issue or a matter of adjusting the label settings, but the labels generated where there are overlapping points are not displayed correctly.
In the example below, I have 2 points at the same location, but I cannot seem to find a way to move the individual labels independently. When I move the label, all overlapping labels are moved at the same time.

I get this if source layers have ruled based styles, categorized or any complex renderer