fix: update CD maps#957
Conversation
lenderom
added
content
|
Hi there! 👋 Thank you for your contribution to the FIP Guide! 🚀 Checklist before merging:
|
✅ Deploy Preview for fipguide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| {{< float-image | ||
| src=`["cd-fip-map-overview-26.webp","cd-fip-map-detail-26.webp"]` | ||
| alt=`["FIP Gültigkeitskarte Überblick für Tschechien","FIP Gültigkeitskarte Detail für Tschechien"]` | ||
| caption=`["[Überblick FIP Gültigkeitskarte (Vollbild)](/en/operator/cd/cd-fip-map-overview-26.webp) für Tschechien (© Petr Duda)","[Detail FIP Gültigkeitskarte (Vollbild)](/en/operator/cd/cd-fip-map-detail-26.webp) für Tschechien (© Petr Duda)"]` | ||
| width="100%" | ||
| position="left">}} | ||
| {{< /float-image >}} |
There was a problem hiding this comment.
Danke dir fürs Anpassen – ich persönlich bin mir unsicher, ob diese Syntax so verständlich ist. Werden wir das an anderer Stelle wiederverwenden? Die Frage ist auch, ob wir es nicht anders als mit Float lösen können, um zwei Bilder nebeneinander darzustellen. 🤔 Lass uns da gern nochmal zu quatschen :)
There was a problem hiding this comment.
Jup gerne. War auch nur eine kurze Idee. WIr können da aber auch einfach Custom HTML injecten fürs Erste.
There was a problem hiding this comment.
Ich mag den Syntax auch nicht wirklich, inbesondere durch die erhöhte Komplexität. Und so ganz verstehe ich den Sinn auch noch nicht. Eigentlich ist das float-image nämlich dafür gedacht, dass Text um das Image herum "schweben" kann. Das wird hier aber gar nicht genutzt, da das shortcode self-closing ist.
Wenn es lediglich darum geht, zwei Bilder parallel anzuzeigen, geht das auch per custom markdown attributes. Hierfür müssten wir einfach die Attributes im render-image hook abrufen, das kann ich gleich mal als separaten PR aufmachen.
Alternativlösungen, die meiner Meinung nach aber wesentlich schlechter geeignet sind:
- Einfach float-image nesten. Zu beachten ist, dass das innere float-image eine relative Positionsberechnung hat, also in diesem Fall hätte das parent 50% width und das innere 100%. Aber da es hier kein float-image ist, gefällt mir das nicht sonderlich gut.
- Wir führen ein column shortcode ein, das die width des entsprechenden containers beschränkt.
- Custom HTML wie gerade, dort funktioniert das jedoch image processing nicht
- Ein flex shortcode, there ein wrapper als flexbox anbietet.
There was a problem hiding this comment.
Mit der angepassten Version im HTML können wir es dennoch mergen, solange wir es als issue festhalten.
There was a problem hiding this comment.
Die Lösung per markdown attributes funktioniert leider doch nicht. Ich habe mal den column shortcode implementiert, da er sicher auch noch für andere Fälle hilfreich sein kann, wenn wir mehrere columns benötigen: #970
Aber können es sonst auch nochmal in die nächste Sitzung mitnehmen.
3 participants
Fixes #747
Replace the CD validity map block in DE/EN/FR with inline HTML so both maps are shown side by side at equal width, each with its own full-view link.