Improve ImageRead widget formatting

[jsouter]

Closes #74

Works, will work on improving how to configure the size of the image widget.

[codecov]

Codecov Report

❌ Patch coverage is 93.75000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.20%. Comparing base (1763528) to head (010f326).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/pvi/_format/bob.py	92.85%	2 Missing ⚠️
src/pvi/_format/screen.py	90.90%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
+ Coverage   91.12%   91.20%   +0.07%     
==========================================
  Files          23       23              
  Lines        1465     1523      +58     
==========================================
+ Hits         1335     1389      +54     
- Misses        130      134       +4     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[GDYendell]

This currently only converts ImageReads at the root of a screen. I think if you move the conversion into generate_component_formatters it should work in groups as well.

[jsouter]

WIP, there are customisation options like enabling color_bars that I'm not sure if they should belong to the dls.bob template (and parsed by the WidgetFormatter) or should be widget options (so FastCS can set them).

I also changed an inconsistency about whether subscreen buttons have labels in groups or outside of groups, not sure which is preferred.
Looks like it also affects label widgets, so will probably have to fix that

[jsouter]

A:

B:

[GDYendell]

I wonder can have option B, but with some kind of "open on new screen" icon as the button text? This is the best icon I could find...

(D050000L font p)

@coretl ?

[coretl]

I think that icon is good. I have no strong opinion between A and B, but I'd like to see that icon at the end of any button text that launches a new screen

[GDYendell]

I don't think the label text can be the signal name + the icon on the end because to get that icon you have to set the button label font to one that is all just icons.

For now I suggest we go with B and the duplicated label and make a new Issue to use that icon for all buttons that open sub screens.

[GDYendell]

I think this is doing what we want now. It would be nice to test with live PVs.

[GDYendell]

I have tested this with fastcs-eiger using the pva transport and it works nicely for the monitor stream. I haven't tested the array trace, we can use CatIO to try this but I am going to merge this for now.

[jsouter]

I have tested this with fastcs-eiger using the pva transport and it works nicely for the monitor stream. I haven't tested the array trace, we can use CatIO to try this but I am going to merge this for now.

Sounds good, I tested this previously with live PVs with a branch of FastCS, will give it a try now

[jsouter]

e.g.