Error text overflow

[mollykreis]

Pull Request

🤨 Rationale

Fixes #2431

👩‍💻 Implementation

The shared error pattern has been updated to use the overflow directive to track when the error text has overflow and to only show the title when overflow is present. As part of this, each component that implements ErrorPattern was updated to include an observable property named errorHasOverflow to correctly implement the ErrorPattern interface.

🧪 Testing

Manually tested in storybook that all updated components:

• don't have a title on the error text when the error text is short
• have a title on the error text when the error text overflows
• the overflow on the error text does not affect any existing overflow behavior on the component

✅ Checklist

• I have updated the project documentation to reflect my changes or determined no changes are needed.

[mollykreis]

@m-akinc, will you buddy this PR for me?

[m-akinc]

Could/should we replace the ErrorPattern interface with a mixin? Would be nice to remove all the declarations of those three properties and automatically enforce the @attr and @observable expectations.

[m-akinc]

Probably worthwhile to write unit tests for this behavior against one (not all) of the components that use this pattern. I'm pretty sure we have some existing title overflow behavior tests for labels, etc.