Replace Option<Component<T>> with Component<Option<T>>

[LucasPickering]

Description

Describe the change. If there is an associated issue, please include the issue link (e.g. "Closes #xxx"). For UI changes, please also include screenshots.

This makes code with optional data fields less clunky (hopefully). Specifically it eliminates the need for weird iter/option flattening in children impls, and if let guards for drawing optional data.

Known Risks

What issues could potentially go wrong with this change? Is it a breaking change? What have you done to mitigate any potential risks?

Code could potentially be harder to follow/more magical

QA

How did you test this?

Checklist

• Have you read CONTRIBUTING.md already?
• Did you update CHANGELOG.md?
  • Only user-facing changes belong in the changelog. Internal changes such as refactors should only be included if they'll impact users, e.g. via performance improvement.
• Did you remove all TODOs?
  • If there are unresolved issues, please open a follow-on issue and link to it in a comment so future work can be tracked