Skip to content

Fix location component error messages#243

Merged
davidjamesstone merged 6 commits intomainfrom
fix/DF-529-location-fixes
Nov 13, 2025
Merged

Fix location component error messages#243
davidjamesstone merged 6 commits intomainfrom
fix/DF-529-location-fixes

Conversation

@davidjamesstone
Copy link
Copy Markdown
Contributor

@davidjamesstone davidjamesstone commented Nov 11, 2025

Proposed change

  • Fix error messages for location components to use short description (with lowerFirst)
  • Changed OsGridRef (NGR) to support 6, 8 or 10 digits
  • Removed NGFR 10 digits
  • Fix error messages
  • Removed custom CSS

Jira ticket: https://eaflood.atlassian.net/browse/DF-529

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Misc. (documentation, build updates, etc)

Checklist

  • You have executed this code locally and it performs as expected.
  • You have added tests to verify your code works.
  • You have added code comments and JSDoc, where appropriate.
  • There is no commented-out code.
  • You have added developer docs in README.md and docs/* (where appropriate, e.g. new features).
  • The tests are passing (npm run test).
  • The linting checks are passing (npm run lint).
  • [x The code has been formatted (npm run format).

mokhld and others added 2 commits November 13, 2025 09:52
* feat: errors inline on Location components

- Improved error message handling in `LocationFieldHelpers` by including `errorMessage` in the view model.
- Modified HTML template to display error messages and adjust form group classes based on error presence.

* lint: fixes

* test: update tests

* fix: remove unnecessary whitespace in location field template

* style: Updated flex properties for better layout on desktop and mobile views.

* style: refactor location input styles and HTML structure for improved error handling and layout

- Updated SCSS for error states in location input fields, ensuring consistent padding and margin.
- Refactored HTML structure to utilize grid layout for better responsiveness and alignment of input items.

* style: add margin-top to location input grid row for improved spacing

* style: remove unused location input styles

* Remove two thirds wrapper

* style: remove unused location input imports from references

---------

Co-authored-by: David Stone <davidjamesstone@gmail.com>
@sonarqubecloud
Copy link
Copy Markdown

@davidjamesstone davidjamesstone merged commit fa7a818 into main Nov 13, 2025
9 checks passed
@davidjamesstone davidjamesstone deleted the fix/DF-529-location-fixes branch November 13, 2025 10:05
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