Skip to content

Comments

Add URL parameter support and fix currency symbol display#5

Merged
petre-c merged 1 commit intomainfrom
feat/url-params-currency-symbols
Jul 13, 2025
Merged

Add URL parameter support and fix currency symbol display#5
petre-c merged 1 commit intomainfrom
feat/url-params-currency-symbols

Conversation

@petre-c
Copy link
Contributor

@petre-c petre-c commented Jul 13, 2025

Summary

  • Add support for URL-based currency input in the format /currencyname/amount (e.g., /eur/352.35)
  • Fix currency symbol rendering issues that caused symbols to not display on desktop browsers
  • Improve cross-platform compatibility for Bitcoin (₿) and Georgian Lari (₾) symbols

Changes

  • Added route parameter support to Index.razor for /currency/amount format
  • Removed flag emojis (🇺🇸, 🇪🇺, 🇬🇧) that don't render on Windows desktop
  • Added CSS classes and font fallbacks for currency symbols
  • Implemented JavaScript detection for unsupported symbols
  • Added text fallbacks (BTC for ₿, GEL for ₾) when symbols aren't supported

Test plan

  • Test URL navigation with different currencies: /eur/100, /usd/50, /gbp/200
  • Verify currency symbols display correctly on Windows, macOS, and Linux
  • Check that fallback text (BTC/GEL) appears when symbols aren't supported
  • Confirm mobile browsers still display symbols correctly
  • Test that root URL / still works with default values

Collaboration by Claude

- Add support for URL format /currency/amount (e.g., /eur/352.35)
- Fix currency symbol rendering issues on desktop browsers
- Remove flag emojis that don't display on Windows
- Add font fallbacks for Bitcoin (₿) and Georgian Lari (₾) symbols
- Implement JavaScript detection for unsupported symbols
- Add text fallbacks (BTC/GEL) when symbols aren't supported

*Collaboration by Claude*
@github-actions
Copy link

✅ Build succeeded! The Blazor WASM app compiled successfully.

@petre-c petre-c merged commit b9471c4 into main Jul 13, 2025
1 check passed
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.

1 participant