entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 9 Releases Wiki Activity

Publik Famille Démo : intégration des pictos (#72951) #103

Merged
csechet merged 4 commits from wip/72951-publik-famille-demo-integration- into main 2023-02-01 18:57:18 +01:00
Conversation 3 Commits 4 Files Changed 25 +115 -70
csechet commented 2023-01-04 10:40:00 +01:00
Owner
Copy Link
No description provided.
csechet changed title from Publik Famille Démo : intégration des pictos to Publik Famille Démo : intégration des pictos (#72951) 2023-01-04 10:41:42 +01:00
csechet changed title from Publik Famille Démo : intégration des pictos (#72951) to WIP: Publik Famille Démo : intégration des pictos (#72951) 2023-01-04 12:05:33 +01:00
csechet force-pushed wip/72951-publik-famille-demo-integration- from 62ab69b006 to e8773c82e3 2023-01-09 16:53:22 +01:00 Compare
csechet changed title from WIP: Publik Famille Démo : intégration des pictos (#72951) to Publik Famille Démo : intégration des pictos (#72951) 2023-01-09 16:53:37 +01:00
csechet commented 2023-01-09 16:56:25 +01:00
Author
Owner
Copy Link

On utilisait l'image de la page pour les icônes, j'ai utilisé des classes custom déclarées dans le CSS pour le même résultat, pour ne pas trop pirater le Core.

On utilisait l'image de la page pour les icônes, j'ai utilisé des classes custom déclarées dans le CSS pour le même résultat, pour ne pas trop pirater le Core.
tjund requested review from tjund 2023-01-25 16:58:18 +01:00
tjund commented 2023-01-26 10:24:09 +01:00
Owner
Copy Link

Retour suite Tests.

pf-icon:

  • un :hover sur l'icon active l'animation, mais l'icon n'est pas cliquable, assez troublant niveau UI
  • l'alignement icon / label est parfois étrange lorsque le label est sur 2 lignes

pf-icon-button

  • le layout casse lorsque le label passe sur plusieurs lignes
  • pas de survol

Globalement, les class ont une utilisation assez specifique : pf-icon uniquement sur items de liste de liens, pf-icons-button uniquement sur lien au sein d'une cellule texte. Ces class pourrait être utilisées sur des cellules liens aussi par exemple. Mais ça peut faire l'objet d'un autre ticket.

Retour suite Tests. pf-icon: * un :hover sur l'icon active l'animation, mais l'icon n'est pas cliquable, assez troublant niveau UI * l'alignement icon / label est parfois étrange lorsque le label est sur 2 lignes pf-icon-button * le layout casse lorsque le label passe sur plusieurs lignes * pas de survol Globalement, les class ont une utilisation assez specifique : pf-icon uniquement sur items de liste de liens, pf-icons-button uniquement sur lien au sein d'une cellule texte. Ces class pourrait être utilisées sur des cellules liens aussi par exemple. Mais ça peut faire l'objet d'un autre ticket.
csechet added 3 commits 2023-01-30 12:08:43 +01:00
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details
42c040b3e9 fix layout of pf-icon-button on multiple lines & underline on hover
csechet commented 2023-01-30 12:10:21 +01:00
Author
Owner
Copy Link

Globalement, les class ont une utilisation assez specifique : pf-icon uniquement sur items de liste de liens, pf-icons-button uniquement sur lien au sein d'une cellule texte. Ces class pourrait être utilisées sur des cellules liens aussi par exemple. Mais ça peut faire l'objet d'un autre ticket.

D'accord avec ça, je propose de voir avec Stef ce dont il a besoin et d'étendre à d'autres utilisations si besoin à la suite de ses retours.

> Globalement, les class ont une utilisation assez specifique : pf-icon uniquement sur items de liste de liens, pf-icons-button uniquement sur lien au sein d'une cellule texte. Ces class pourrait être utilisées sur des cellules liens aussi par exemple. Mais ça peut faire l'objet d'un autre ticket. > D'accord avec ça, je propose de voir avec Stef ce dont il a besoin et d'étendre à d'autres utilisations si besoin à la suite de ses retours.
tjund approved these changes 2023-02-01 18:23:17 +01:00
csechet merged commit f80201448e into main 2023-02-01 18:57:18 +01:00
csechet deleted branch wip/72951-publik-famille-demo-integration- 2023-02-01 18:57:18 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tjund
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#103
No description provided.
Delete Branch "wip/72951-publik-famille-demo-integration-"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?