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

Conflitc with QControls #2

Open
BOBILLIER opened this issue Mar 22, 2021 · 11 comments
Open

Conflitc with QControls #2

BOBILLIER opened this issue Mar 22, 2021 · 11 comments

Comments

@BOBILLIER
Copy link

Hi François
Today I have encountered one issue with your "Create customized control" tool. I have run it on LV2011 and it's ok, but under LV2018 it's broken. It seems there is a class name conflict with your Control.lvclass and the same in QControl tools
I have to try to rename your class, but like it's locked, it's not so easy. Have you any suggestions to solve this?
I send
Class conflit with QControl
a message to QControls support to indicate that I have to find this problem.

@francois-normandin
Copy link
Member

francois-normandin commented Mar 22, 2021

@BOBILLIER I think you'll need to uninstall QControls first (to unlock the UI Tools class) and then change the class name in UI Tools. Finally, when all is relinked to the new namespace, you can reinstall QControls and wouldn't get a conflict.

Of course, the real way to solve the issue would be to put the UI Tools methods inside a lvlib and provide a permanent fix.

@francois-normandin
Copy link
Member

@BOBILLIER
I created a 2.0.0 package where namespace has been changed to avoid name collision with QControls
This is not a full release in the sense that I will try to find time to clear out a few quirks and re-release fully through VIPM Community network.

For now, please find the VIP file here:
https://github.com/LabVIEW-Open-Source/ui-tools-controls-addon/releases/tag/2.0.0

J'espère que ça fonctionnera bien. J'ai testé en LV2020, mais sans QControls... sauf que je ne vois pas comment ça causerait problème!

@BOBILLIER
Copy link
Author

BOBILLIER commented Mar 23, 2021 via email

@francois-normandin
Copy link
Member

Hmmm, j'aurais cru que l'ancienne version serait désinstallée en installant la nouvelle. Il y a dû y avoir un changement de nom de la librairie quand j'ai recompilé la librairie de base. J'ai des deadline pas mal serrés aujourd'hui alors pourquoi de chance que je puisse y faire quelque chose immédiatement, mais ça m'aiderait d'avoir une vue depuis VIPM pour me montrer quelles versions de UITools sont installées.

Pour les boutons avec un nouveau modèle, je crois qu'il y a une vieille vidéo sur LAVAG, dans le dépôt de code, à ce sujet. Je devrais en refaire une nouvelle.
En gros, il fait que les nouvelles images (PNG) soient identiques aux états dans le bouton qui se trouve dans le même répertoire. Si une image est associée à l'état "enfoncé", alors l'image que utilisée pour créer cet état doit être identique à celle trouvée dans le répertoire.

Je pourrais fonctionner avec un fichier qui décrit quelle image va avec quel état, mais c'était plus simple de comparer le modèle en extrayant les images déjà contenues et en les remplaçant par les modèles, auxquelles le programme a apposer l'icône.

@BOBILLIER
Copy link
Author

Voici une copie d'écran de VIPM
image
Par contre pour mon contrôle, j'ai mis à jour le contrôle avec les images du même dossier et cela ne fonctionne toujours pas (pas de preview) . Je te joins un zip avec les éléments, si tu veux tester
NXG.zip

@francois-normandin
Copy link
Member

Ça semble venir de la librairie add-on control 1.2.
C'est essentiellement la même que j'ai mis à jour, donc c'est que j'ai effectivement un changement de nom de package. Je vais y voir.

Pour le bouton, je regarde ça en arrivant au bureau. Ça prend normalement quelques minutes à faire...

@francois-normandin
Copy link
Member

francois-normandin commented Mar 23, 2021

@BOBILLIER
Wow, I need to make this simpler... :-)
You need to have one image per state... and the name of the image needs to be "Camel Case"...
Nxg.zip

The folder name can be capitalized, but the images and their links need to be following a strict rule.
I'll definitely rework this. I don,t know whey this choice was made 10 years ago... but it is not scalable to my liking!

Proof it now works:
image

@francois-normandin
Copy link
Member

Install them under vilib>LAVA folder.

image

@francois-normandin
Copy link
Member

@BOBILLIER
Est-ce que j'ai ta permission pour inclure ton template NXG dans le prochain build?

@BOBILLIER
Copy link
Author

BOBILLIER commented Mar 23, 2021 via email

@francois-normandin
Copy link
Member

You are right... states and number of images are not supposed to be linked.
In this case, the errors I was seeing were to that effect where there were empty paths.
Anyhow, I'll rework this for the next round. If I release a version 2.0.0 officially, I want it to be an improved version of the control generator!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants