Skip to content

Comments

UHF-11878: Added nowrap for calculator sum#1470

Merged
annadruid merged 3 commits intomainfrom
UHF-11878
Jan 9, 2026
Merged

UHF-11878: Added nowrap for calculator sum#1470
annadruid merged 3 commits intomainfrom
UHF-11878

Conversation

@annadruid
Copy link
Contributor

@annadruid annadruid commented Jan 9, 2026

UHF-11878

What was done

  • Added white space no wrap for calculator sums

How to install

  • Make sure your instance is up and running on latest dev-branch
    • git checkout dev && git pull origin dev
    • make fresh
  • Update the HDBT theme
    • composer require drupal/hdbt:dev-UHF-11878
  • Run code updates
    • make drush-locale-update drush-cr

How to test

  •  See platform config pr
  • Check that the code follows our standards
  • Check that the differences in visual regression tests are appropriate

Links to related PRs

Copilot AI review requested due to automatic review settings January 9, 2026 06:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds CSS styling to prevent text wrapping in calculator sum displays by applying white-space: nowrap to the .helfi-calculator__receipt-subtotal-sum class.

  • Added white-space: nowrap property to calculator subtotal sum elements

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@teroelonen teroelonen left a comment

Choose a reason for hiding this comment

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

This is good 🦖

@github-actions
Copy link

github-actions bot commented Jan 9, 2026

⚠️ 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/1470/html_report/

@annadruid annadruid merged commit ed87c4d into main Jan 9, 2026
6 checks passed
@annadruid annadruid deleted the UHF-11878 branch January 9, 2026 12:27
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