Saving changes in Settings UI changes Precision for Decimal Values in settings.json #12376

Closed
opened 2026-01-31 03:13:58 +00:00 by claunia · 5 comments
Owner

Originally created by @ShubhamBelwal on GitHub (Feb 2, 2021).

Environment

Windows build number: Microsoft Windows [Version 10.0.19041.746]
Windows Terminal version (if applicable): Windows Terminal Preview v1.6.10272.0

Any other software? NA

Steps to reproduce

  1. Note decimal values for fields acrylicOpacity & backgroundImageOpacity
  2. Open Settings UI, make some change & click on Save
  3. Observe that precision for the values have changed.

Expected behavior

The precision for the decimal values should not change

Actual behavior

Values now have precision of 17 digits instead of 2.

Before:
image

After:
image

Originally created by @ShubhamBelwal on GitHub (Feb 2, 2021). # Environment ```none Windows build number: Microsoft Windows [Version 10.0.19041.746] Windows Terminal version (if applicable): Windows Terminal Preview v1.6.10272.0 Any other software? NA ``` # Steps to reproduce 1. Note decimal values for fields `acrylicOpacity` & `backgroundImageOpacity` 2. Open Settings UI, make some change & click on Save 3. Observe that precision for the values have changed. # Expected behavior The precision for the decimal values should not change # Actual behavior Values now have precision of 17 digits instead of 2. Before: ![image](https://user-images.githubusercontent.com/36201797/106646359-a8c58380-65b3-11eb-9321-9b13fddc8fb8.png) After: ![image](https://user-images.githubusercontent.com/36201797/106646301-96e3e080-65b3-11eb-84d5-e329c573d243.png)
claunia added the Resolution-Duplicate label 2026-01-31 03:13:58 +00:00
Author
Owner

@DHowett commented on GitHub (Feb 2, 2021):

So, this is only half correct. The precision isn't changing, it's just being exposed to you. IEEE954 rules!

It is annoying though.

@DHowett commented on GitHub (Feb 2, 2021): So, this is only half correct. The precision _isn't changing_, it's just being exposed to you. IEEE954 rules! It _is_ annoying though.
Author
Owner

@zadjii-msft commented on GitHub (Feb 3, 2021):

Should we just include this with #8991 for ""settings UI reformats my file?

@zadjii-msft commented on GitHub (Feb 3, 2021): Should we just include this with #8991 for ""settings UI reformats my file?
Author
Owner

@DHowett commented on GitHub (Feb 3, 2021):

Yes, that would be good.

@DHowett commented on GitHub (Feb 3, 2021): Yes, that would be good.
Author
Owner

@DHowett commented on GitHub (Feb 4, 2021):

/dup #8991

@DHowett commented on GitHub (Feb 4, 2021): /dup #8991
Author
Owner

@ghost commented on GitHub (Feb 4, 2021):

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost commented on GitHub (Feb 4, 2021): Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#12376