openmrs / openmrs-esm-form-engine-lib

React Form Engine library for O3
Other
10 stars 59 forks source link

(fix) Add hide expression condition for number fields #370

Closed pirupius closed 1 month ago

pirupius commented 1 month ago

Requirements

Summary

This adds condition to hide number input if the hide expression is true. Currently the hideExpression doesn't work for number fields and those fields are always rendered. This fixes that.

Screenshots

Before Number fields are always rendered regardless of expression logic

Screenshot 2024-09-03 at 14 04 29

Expected Behavior Number fields with hide expression logic should be hidden by default and only visible when expression condition is true

Screenshot 2024-09-03 at 14 07 13

Related Issue

Other

github-actions[bot] commented 1 month ago

Size Change: -18 B (0%)

Total Size: 1.14 MB

ℹ️ View Unchanged | Filename | Size | Change | | :--- | :---: | :---: | | `dist/151.js` | 300 kB | 0 B | | `dist/225.js` | 2.57 kB | 0 B | | `dist/277.js` | 1.84 kB | 0 B | | `dist/3.js` | 481 B | 0 B | | `dist/300.js` | 642 B | 0 B | | `dist/335.js` | 968 B | 0 B | | `dist/353.js` | 3.02 kB | 0 B | | `dist/41.js` | 3.37 kB | 0 B | | `dist/422.js` | 6.8 kB | 0 B | | `dist/540.js` | 2.63 kB | 0 B | | `dist/55.js` | 758 B | 0 B | | `dist/572.js` | 252 kB | -13 B (-0.01%) | | `dist/617.js` | 86.9 kB | 0 B | | `dist/635.js` | 14.3 kB | 0 B | | `dist/733.js` | 107 kB | 0 B | | `dist/901.js` | 11.8 kB | 0 B | | `dist/99.js` | 691 B | 0 B | | `dist/993.js` | 3.09 kB | 0 B | | `dist/main.js` | 340 kB | -5 B (0%) | | `dist/openmrs-esm-form-engine-lib.js` | 3.67 kB | 0 B |

compressed-size-action