Contact summary - adjust block width when editing

[colemanw]

Overview

Fixes an annoyance when editing blocks on the contact summary screen where their content would be hidden with a scrollbar.

Before

Forms that were too wide would be cropped and need to be manually scrolled.

After

Forms will auto-adjust their width to fit content.

Notes

Compatible with Shoreditch and Contact Summary Layout Editor extensions.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[tschuettler]

That looks and feels very nice. 💚

Fields with multiple values like phone look a bit different from your screencap. They have a visible empty cell on the right that is used for the delete button and are not centered.

Adding a second phone number will not increased the width enough to fully display the delete button:

There should probably a maximum width, just in case...

[colemanw]

@tschuettler thanks for the review. I've tweaked the code to rely more on css and less on javascript, and the result is more flexible. It now wraps help text correctly, responds to width changes when the form needs to expand to show something like a delete button, and responds pretty well to the window being resized mid-edit. Give it another try :)

[tschuettler]

Thanks for the additional tweaks. I gave it another try to break stuff and it is looking good to me.

There is now a scrollbar when the fieldset gets too wide and is on the left side 👍 :

For the right hand side custom field sets there does not appear a scroll bar once the field gets to large for the viewport. Anyway, it might be more sensible to use the options per line setting to adjust for that :)

[colemanw]

Thanks again for the thorough testing @tschuettler! Glad you haven't been able to break it :)
IMO the "lots of radios on one line" is a preexisting condition. It looked awful before this PR, still looks pretty bad after (slightly better IMO) and would be a good thing to improve on once this PR gets merged.
@lcdservices care to weigh in?

[lcdservices]

@colemanw I implemented and tested in shoreditch -- looks great. I think this is a big improvement.