Labels Milestones

WIP icones_svg #1

Merged

b_b merged 10 commits from :icones_svg into master 4 years ago

Conversation 15 Commits 10 Files Changed 83

tcharlss commented 4 years ago

Owner

Ticket 4406 : réfection du style et des icônes de la barre d'édition (passage en svg).

[Ticket 4406](https://core.spip.net/issues/4406) : réfection du style et des icônes de la barre d'édition (passage en svg). 

👍 5

b_b commented 4 years ago

Owner

Super, premier commentaire, pourquoi garder des règles inutilisées dans des commentaires comme ici 41f5be0fd5/css/barre_outils.css (L68) ?

À part ça, gogogo ça fera un truc de plus dans la 3.3 :)

Super, premier commentaire, pourquoi garder des règles inutilisées dans des commentaires comme ici https://git.spip.net/tcharlss/porte_plume/src/commit/41f5be0fd5fd66f6df9ebf033862f2f50d4bddd8/css/barre_outils.css#L68 ? À part ça, gogogo ça fera un truc de plus dans la 3.3 :)

tcharlss commented 4 years ago

Poster

Owner

Il y avait déjà quelques règles commentées avant, comme ici par exemple : 41f5be0fd5/css/barre_outils.css (L3-L4)

Du coup j'ai fait pareil pour celles que j'ai désactivées, jusqu'au cas où.
Mais oui, ça peut être enlevé, après tout c'est versionné tout ça.

Il y avait déjà quelques règles commentées avant, comme ici par exemple : https://git.spip.net/tcharlss/porte_plume/src/commit/41f5be0fd5fd66f6df9ebf033862f2f50d4bddd8/css/barre_outils.css#L3-L4 Du coup j'ai fait pareil pour celles que j'ai désactivées, jusqu'au cas où. Mais oui, ça peut être enlevé, après tout c'est versionné tout ça.

👍 1

cerdic commented 4 years ago

Owner

Si ça pète pas le plugin frimousses c'est top en plus (parce que j'ai du bidouiller avec le porte plume sur celui là)

Si ça pète pas le plugin frimousses c'est top en plus (parce que j'ai du bidouiller avec le porte plume sur celui là)

tcharlss commented 4 years ago

Poster

Owner

Ça a l'air bon avec frimousses :

Ça a l'air bon avec frimousses : 

barre_svg_frimousses.png

40 KiB

RealET commented 4 years ago

Et ça donne quoi avec les enluminures typographiques ?
https://contrib.spip.net/Enluminures-typographiques-V3-3118

Et ça donne quoi avec les enluminures typographiques ? https://contrib.spip.net/Enluminures-typographiques-V3-3118

nicod commented 4 years ago

Owner

Et ça donne quoi avec les enluminures typographiques ?
https://contrib.spip.net/Enluminures-typographiques-V3-3118

Tu testes et tu nous dis ?

> Et ça donne quoi avec les enluminures typographiques ? > https://contrib.spip.net/Enluminures-typographiques-V3-3118 Tu testes et tu nous dis ?

tcharlss commented 4 years ago

Poster

Owner

Avec les enluminures il manque 2 icônes : intertitre niveau 1 et texte barré

Avec les enluminures il manque 2 icônes : intertitre niveau 1 et texte barré  

barre_svg_enlum1.png

18 KiB

barre_svg_enlum2.png

19 KiB

tcharlss commented 4 years ago

Poster

Owner

@RealET

Les 2 icônes manquantes sont des icônes "vanilla" qui sont réutilisées, ors j'ai ajusté les dimensions du sprite, les coordonnées ont donc changé.

Il faudrait donc mettre à jour les coordonnées de ces 2 entrées dans enluminures, cf. https://git.spip.net/tcharlss/porte_plume/src/branch/icones_svg/barre_outils/edition.php#L491 et https://git.spip.net/spip-contrib-extensions/porte_plume_enluminures_typographiques/src/branch/master/enluminures_typo_pipelines.php#L229

@RealET Les 2 icônes manquantes sont des icônes "vanilla" qui sont réutilisées, ors j'ai ajusté les dimensions du sprite, les coordonnées ont donc changé. Il faudrait donc mettre à jour les coordonnées de ces 2 entrées dans enluminures, cf. https://git.spip.net/tcharlss/porte_plume/src/branch/icones_svg/barre_outils/edition.php#L491 et https://git.spip.net/spip-contrib-extensions/porte_plume_enluminures_typographiques/src/branch/master/enluminures_typo_pipelines.php#L229

tcharlss commented 4 years ago

Poster

Owner

En nb de +1 ça à l'air bon. Des objections à fusionner ? Je préfère demander une dernière fois avant ^^

En nb de +1 ça à l'air bon. Des objections à fusionner ? Je préfère demander une dernière fois avant ^^

tcharlss commented 4 years ago

Poster

Owner

@RealET je te ferais un patch pour enluminures si tu veux

@RealET je te ferais un patch pour enluminures si tu veux

👍 1

b_b commented 4 years ago

Owner

On squash ou on intègre les 10 commits ?

On squash ou on intègre les 10 commits ?

tcharlss commented 4 years ago

Poster

Owner

je dirais qu'il vaut mieux squasher car il y a eu pas mal d'allers-retours

je dirais qu'il vaut mieux squasher car il y a eu pas mal d'allers-retours

b_b referenced this issue from a commit 4 years ago

icones svg pour le porte plume (!1) Icône svg un peu plus fidèle à l'ancienne Tant qu'à faire, une mignonnette icône en svg + les crédits dans paquet.xml Changement pour l'italique : i → I. Finitions dans le CSS. On supprime les vieux png, remplacés par les svg. Fallback pour les variables CSS «G» pour gras, «T» pour intertitres, couleur espace privé en amélioration progressive, handle en svg aussi Crédits pour certaines icônes Màj icônes et réorganisation du fichier. Optimisation svg svg avec les textes transformés en chemins un premier jet, à affiner Co-authored-by: Charles Razack <tcharlss@bravecassine.com>

b_b closed this pull request 4 years ago

RealET commented 4 years ago

@RealET je te ferais un patch pour enluminures si tu veux
Je veux bien, merci !
Et si possible, sans avoir 2 branches, une pour 3.3 et une pour avant.

Merci d'avance.

> @RealET je te ferais un patch pour enluminures si tu veux Je veux bien, merci ! Et si possible, sans avoir 2 branches, une pour 3.3 et une pour avant. Merci d'avance.

tcharlss deleted branch icones_svg 4 years ago

tcharlss commented 4 years ago

Poster

Owner

Je veux bien, merci !
Et si possible, sans avoir 2 branches, une pour 3.3 et une pour avant.

Oui ça devrait être jouable

> Je veux bien, merci ! > Et si possible, sans avoir 2 branches, une pour 3.3 et une pour avant. Oui ça devrait être jouable

❤️ 1

RealET commented 4 years ago

Je ne suis pas sûr que T soit bien indiqué :

1. ça va obliger à former les gens pour le dire que c'est plus H, mais T (rugosité au changement)
2. ici, dans Gitea, c'est aussi H...
3. et dans le code HTML, ça produit Hn

>  Je ne suis pas sûr que T soit bien indiqué : 1. ça va obliger à former les gens pour le dire que c'est plus H, mais T (rugosité au changement) 1. ici, dans Gitea, c'est aussi H... 1. et dans le code HTML, ça produit Hn

The pull request has been merged as 388c1bcc13.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels   

amélioration

Amélioration, nouvelle fonctionnalité   

bug

Ca ne fonctionne pas   

documentation

Documentation souhaite   

doublon

Ce ticket est un doublon   

invalide

Ticket invalide   

refusé

Ignoré, c'est comme Ca...

  

à confirmer

No Label

amélioration

bug

documentation

doublon

invalide

refusé

à confirmer

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

5 Participants

Notifications

Subscribe

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: spip/porte_plume#1

Write Preview

Loading…

Cancel

Save

There is no content yet.