Skip to content

fix(number): replace Number with parseFloat for inline width parsing and remove Number #7603#8027

Merged
deleonio merged 1 commit intopublic-ui:developfrom
donchi-donald:7603-avoid-using-Number-instead-of-parseInt-parseFloat
Jul 15, 2025
Merged

fix(number): replace Number with parseFloat for inline width parsing and remove Number #7603#8027
deleonio merged 1 commit intopublic-ui:developfrom
donchi-donald:7603-avoid-using-Number-instead-of-parseInt-parseFloat

Conversation

@donchi-donald
Copy link
Copy Markdown
Contributor

Beschreibung der Änderungen

Im Rahmen des Tickets #7603 wurden gezielt zwei Stellen im Code überarbeitet, um die Verwendung von Number() zu vermeiden und durch geeignetere Alternativen zu ersetzen:

  1. Formularfeld – toFixed-Anwendung
    Die bisherige Umwandlung Number(field.value).toFixed(2) wurde durch field.value.toFixed(2) ersetzt. Da zuvor bereits geprüft wird, ob field.value vom Typ number ist, war der Einsatz von Number() überflüssig.

  2. CSS-Style – Breitenwert parsen
    Beim Einlesen des width-Werts aus einem CSS-String (z. B. "120px") wurde Number(...) durch parseFloat(...) ersetzt. Dies ermöglicht eine korrekte Verarbeitung auch bei Dezimalwerten

@anicyne anicyne linked an issue Jul 15, 2025 that may be closed by this pull request
2 tasks
@deleonio deleonio merged commit 60a44ea into public-ui:develop Jul 15, 2025
11 checks passed
@publicuibot publicuibot bot locked and limited conversation to collaborators Jul 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Do not use Number instead of parseInt or parseFloat

2 participants