Skip to content

Feat/sprungmarken #46

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

Merged
merged 15 commits into from
Apr 28, 2025
Merged

Feat/sprungmarken #46

merged 15 commits into from
Apr 28, 2025

Conversation

therobrob
Copy link
Member

@therobrob therobrob commented Apr 27, 2025

Description

Task https://github.com/orgs/fipguide/projects/1?pane=issue&itemId=94945248

  • added anchorlinks for all h2-headlines in countries & operator single pages
  • added click-event to copy anchorlink
  • added snackbar (user indicator for successful copy)
  • a11y-features
  • localization

Checklist

  • Changed the date in updatet content pages
  • Check the License of new pictures (non-commercial use without attribution)

The content was modified in the following languages:

  • English
  • German

@therobrob therobrob requested a review from lenderom April 27, 2025 11:17
@lenderom
Copy link
Member

Ging das doch nicht nativ in Hugo? 😅

@lenderom
Copy link
Member

Wollen wir das Link Icon vielleicht immer anzeigen? Also auch wenn es nicht im Focus ist, mit einer niedrigeren Opazität? Ich glaube das könnte Mobil ganz gut sein :)

@therobrob
Copy link
Member Author

therobrob commented Apr 27, 2025

Ging das doch nicht nativ in Hugo? 😅

Immerhin werden die Links nun beim Builden erzeugt und nicht erst zur Laufzeit. Von daher ist es schon "nativ" wie ich finde. :)
Es wohl auch die Option, den Markdown Renderer zu erweitern:

Das habe ich aber erstmal gelassen. Wenn du es hier direkt ausprobieren magst, contribute gern :) Ansonsten können wir dafür gern einen Task für später machen.

@therobrob
Copy link
Member Author

Wollen wir das Link Icon vielleicht immer anzeigen? Also auch wenn es nicht im Focus ist, mit einer niedrigeren Opazität? Ich glaube das könnte Mobil ganz gut sein :)

Können wir machen :)

Copy link
Member

@lenderom lenderom left a comment

Choose a reason for hiding this comment

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

Ich hab mir den MR jetzt nochmal genauer angeschaut. Du hattest das jetzt ja schon genauso implementiert, wie ich mir das dachte. Die Sprungmarken werden ja auch ohne JS erzeugt. Das ist ja nur für die Kopierfunktion zuständig und das könnte man ja auch gar nicht anders lösen :)

@lenderom lenderom merged commit 2d24a9b into main Apr 28, 2025
1 check passed
@therobrob therobrob deleted the feat/sprungmarken branch April 28, 2025 13:55
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.

2 participants