Skip to content

sao: Ensure readonly URL input is visible [CUSTOM]#537

Open
mamoun-rais wants to merge 1 commit intocoopengo:masterfrom
mamoun-rais:fix_url_on_tree_view
Open

sao: Ensure readonly URL input is visible [CUSTOM]#537
mamoun-rais wants to merge 1 commit intocoopengo:masterfrom
mamoun-rais:fix_url_on_tree_view

Conversation

@mamoun-rais
Copy link
Contributor

@mamoun-rais mamoun-rais commented Nov 28, 2025

Fix #PJAZZ-3823
lien vers la fiche : https://coopengo.atlassian.net/browse/PJAZZ-3823

@mamoun-rais
Copy link
Contributor Author

mamoun-rais commented Nov 28, 2025

Pour reproduire le problème :
Il suffit de prendre n’importe quel champ défini en XML avec widget="url" (par exemple le lien sur les pièces jointes) et de le passer en readonly.
Dans ce cas, le lien reste cliquable, mais le texte n’est pas affiché.

Avant le fix

Capture d’écran du 2025-11-28 13-34-48

Après le fix
Capture d’écran du 2025-11-28 13-27-28

@mamoun-rais mamoun-rais changed the title sao: Ensure readonly URL input is visible sao: Ensure readonly URL input is visible [CUSTOM] Nov 28, 2025
@mamoun-rais
Copy link
Contributor Author

@JCavallo @jmousset

Copy link
Member

@jmousset jmousset left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mamoun-rais il faut faire la pr sur la branche 7.6-patches

@mamoun-rais
Copy link
Contributor Author

@mamoun-rais il faut faire la pr sur la branche 7.6-patches

Done : #540

@nicoe
Copy link
Collaborator

nicoe commented Dec 8, 2025

Copy link
Collaborator

@nicoe nicoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comme indiqué dans le lien précédant, l'idéal serait d'indiquer d'une façon ou d'une autre que le lien est cliquable (ou pas). Je n'ai pas trop d'idée de comment faire cela par contre.

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

Successfully merging this pull request may close these issues.

5 participants