feat(span): add setters for content and style

[joshka]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (37c70db) 90.8% compared to head (9087e39) 90.9%.

Additional details and impacted files

@@          Coverage Diff          @@
##            main    #647   +/-   ##
=====================================
  Coverage   90.8%   90.9%           
=====================================
  Files         42      42           
  Lines      12509   12533   +24     
=====================================
+ Hits       11370   11394   +24     
  Misses      1139    1139           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[SOF3]

Actually these should be called "fluent setters" not "builder methods". Builder methods usually refer to a separate builder object that may have incomplete fields and need to be consumed to become the built object, e.g. https://docs.rs/typed-builder.

[joshka]

Actually these should be called "fluent setters" not "builder methods". Builder methods usually refer to a separate builder object that may have incomplete fields and need to be consumed to become the built object, e.g. https://docs.rs/typed-builder.

Good point.

[orhun]

LGTM, as mentioned, we might change it to "fluent setters" before merge.

[Valentin271]

Other than the typo, looks good

[joshka]

Rebased and squashed - should be good to go

[joshka]

Tweaked must_use message to:

    #[must_use = "method moves the value of self and returns the modified value"]

Based on discussion in #655

[Valentin271]

Fine by me, approved