diff options
author | athenian200 <athenian200@outlook.com> | 2020-09-30 14:05:56 -0500 |
---|---|---|
committer | Moonchild <moonchild@palemoon.org> | 2020-10-01 09:55:37 +0000 |
commit | e50094ab02604892ad619abb0e3d1c7c674e43cf (patch) | |
tree | f8a7796c662ac4fce03e94e7aff0d3d9ad768684 /hal | |
parent | d11196927acb017c6ea8a741102906041c447778 (diff) | |
download | UXP-e50094ab02604892ad619abb0e3d1c7c674e43cf.tar UXP-e50094ab02604892ad619abb0e3d1c7c674e43cf.tar.gz UXP-e50094ab02604892ad619abb0e3d1c7c674e43cf.tar.lz UXP-e50094ab02604892ad619abb0e3d1c7c674e43cf.tar.xz UXP-e50094ab02604892ad619abb0e3d1c7c674e43cf.zip |
Issue #1647 - Followup: Remove excessive VARIANT_OPACITY statements.
I got very anxious about making sure I included VARIANT_OPACITY in all the places VARIANT_NUMBER was included to make sure it couldn't possibly break unexpectedly, and that led to me accidentally breaking a mechanism that prevented percentages from serializing as numbers in other parts of the code. It was a total accident, and these additions were unnecessary. Basically, the situation is that there was one part of the code where it determines what's allowed for the flex statement (and possibly other statements) by checking whether it got stored as a "number", and basically only disallows percentages if it attempted to store/serialize them as percentages.
However, it only got to that part of the code because I accidentally allowed VARIANT_OPACITY as a valid way for certain tokens to parse where it wasn't necessary. If it tries to parse it that way under very specific circumstances... percentages will be marked valid and fed through the system as numbers rather than being rejected and not serialized at all, because the check to disallow percentages there relied on them being stored as percentages.
It's a really weird thing to have a problem with in a lot of ways, because if percentages aren't allowed in a field, you would think people wouldn't try to use them there, much less depend on the broken behavior that results from them not parsing as a related value.
Diffstat (limited to 'hal')
0 files changed, 0 insertions, 0 deletions