Bug 2047849 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
### Summary
This meta bug tracks a set of cleanup and refactoring tasks in the onboarding code:

- Remove the `shouldShowElevation` parameter and use the existing `isSmall` flag for gating logic instead.
- Rename the `func` parameter in `toPageUiData` to `jexlEvaluator` for improved clarity.
- Reorder `toPageUiData` parameters to group JEXL-related parameters together.
- Hoist the manufacturer check out of `toPageUiData`, reducing complexity and allowing removal of the `LongParameterList` suppression.
- Reorder private functions in `OnboardingMapper` to follow call order and improve readability.
Revision 1 by
on 
### Summary
This bug tracks a set of clean-up and refactoring tasks in the onboarding code:

- Remove the `shouldShowElevation` parameter and use the existing `isSmall` flag for gating logic instead.
- Rename the `func` parameter in `toPageUiData` to `jexlEvaluator` for improved clarity.
- Reorder `toPageUiData` parameters to group JEXL-related parameters together.
- Hoist the manufacturer check out of `toPageUiData`, reducing complexity and allowing removal of the `LongParameterList` suppression.
- Reorder private functions in `OnboardingMapper` to follow call order and improve readability.

Back to Bug 2047849 Comment 0