Skip to content

UHF-10909 & UHF-9717: Contact card design change, accessibility improvements#1212

Merged
teroelonen merged 10 commits intomainfrom
UHF-10909
Mar 11, 2025
Merged

UHF-10909 & UHF-9717: Contact card design change, accessibility improvements#1212
teroelonen merged 10 commits intomainfrom
UHF-10909

Conversation

@teroelonen
Copy link
Contributor

@teroelonen teroelonen commented Mar 5, 2025

UHF-10909 & UHF-9717

What was done

  • Fixed issues with too small clickable area on announcement.
  • Restyled the person card (contact card) paragraph.
  • Fixed accessibility issues at the same time.

How to install

  • Make sure your instance is up and running on latest dev branch. Use for example ELO instance.
    • git pull origin dev
    • make fresh
  • Update the HDBT theme and the Helfi Platform config
    • composer require drupal/hdbt:dev-UHF-10909 && composer require drupal/helfi_platform_config:dev-UHF-10909
  • Run make drush-updb drush-cr
  • Run make shell
    • In the shell, run drush helfi:platform-config:update helfi_image_styles
    • Then run drush helfi:platform-config:update helfi_paragraphs_contact_card_listing

How to test

Screenshot 2025-03-10 at 16 51 35

Continuous documentation

  • This feature has been documented/the documentation has been updated
  • This change doesn't require updates to the documentation

Other PRs

@annadruid annadruid marked this pull request as ready for review March 6, 2025 05:50
@annadruid annadruid marked this pull request as draft March 6, 2025 05:51
@github-actions
Copy link

github-actions bot commented Mar 6, 2025

⚠️ Visual regression found! Please check if this change is wanted or accidental. You can check the output here: https://city-of-helsinki.github.io/drupal-hdbt-visual-regression/pull/1212/html_report/

@teroelonen teroelonen changed the title UHF-10909 UHF-10909 & UHF-9717 Mar 7, 2025
@teroelonen teroelonen changed the title UHF-10909 & UHF-9717 UHF-10909 & UHF-9717: Contact card design change, accessibility improvements Mar 10, 2025
@teroelonen teroelonen marked this pull request as ready for review March 10, 2025 15:21
@@ -1,3 +1,5 @@
$-announcement_link_padding: 12px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this 12px be a variable? We seem to have $spacing-three-quarters available

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed 🫡

@github-actions
Copy link

⚠️ Visual regression found! Please check if this change is wanted or accidental. You can check the output here: https://city-of-helsinki.github.io/drupal-hdbt-visual-regression/pull/1212/html_report/

@teroelonen teroelonen merged commit 72c5734 into main Mar 11, 2025
2 checks passed
@teroelonen teroelonen deleted the UHF-10909 branch March 11, 2025 07:35
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