Modules: add a panel around diff previews when updating

[mashehu]

fixes #3239

[jfy133]

Do you have a screnshot as to what it looks like?

[mashehu]

👉🏻 #3239 (comment)

[jfy133]

Yes, but it would also be nice to have a marker between each group of files too, e.g. set of files per module.

My reasoning is I need to make sure e.g. I look at the combination of a main.nf and meta to make sure I understand what channel modifications I have to make in the pipeline

[mashehu]

okay, a bit more difficult, but I will give it a try

[jfy133]

okay, a bit more difficult, but I will give it a try

Can just be a straight line across the whole width of screen printed before the question of 'updated X/y/z module'?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.35%. Comparing base (d59ac3f) to head (29f6765).
Report is 12 commits behind head on dev.

Additional details and impacted files

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

[mashehu]

better?

[mashehu]

alternative 2

[jfy133]

Yes I think that will do for now (would need to try to test in the wild a few times ).

The blue border is nice but I find it's a hit harder to read the name of the module

But no strong feeling on the colour

[mashehu]

how about this then

[mirpedrol]

LGTM!