VP1: Add HTML support in labels, related to ATLASVPONE-646

This merge request affects 1 package:

graphics/VP1/VP1Gui

This merge request affects 1 file:

graphics/VP1/VP1Gui/src/VP1EventDisplayEditor.cxx

Adding @rbianchi as watcher

added EventDisplay main review-pending-level-1 labels

added 1 commit

1b1e421f - Update VP1EventDisplayEditor.cxx

Compare with previous version

This merge request affects 1 package:

graphics/VP1/VP1Gui

This merge request affects 1 file:

graphics/VP1/VP1Gui/src/VP1EventDisplayEditor.cxx

Adding @rbianchi as watcher

changed the description

added 24.0 label

changed target branch from main to 24.0

CI Result SUCCESS (hash f8b84c92)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4779]

CI Result SUCCESS (hash 1b1e421f)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4780]

MR looks good to me, approving.

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-approved label

@rbianchi the 24.0 branch is not open for MRs and the branch will be reset early next week (so your changes here will get lost). Please re-target for main.

removed review-approved label

removed main label

added review-user-action-required label

removed 24.0 label

removed review-user-action-required label

added 24.0 label

resolved all threads

Jenkins please retry a build

This merge request affects 1 package:

graphics/VP1/VP1Gui

This merge request affects 1 file:

graphics/VP1/VP1Gui/src/VP1EventDisplayEditor.cxx

Adding @rbianchi as watcher

added review-pending-level-1 label

Hi @rbianchi , @rbianchi ,

It looks from @rbianchi that this does need to target 24.0? If so, since you created it on Friday, I guess this will need to be re-based to synch with 24.0, if indeed, the correct target is 24.0?

Overall it's not 100% clear to me why this needs to be targeting the frozen production branch, unless it's needed for the control room event displays or for debugging 24.0 output?

James

resolved all threads

CI Result FAILURE (hash 1b1e421f)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4865]

resolved all threads

The failing tests do not appear to be related to the changes in this MR

Jenkins please retry a build

MR given back to the developer due to CI test failures.

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-user-action-required label

@yvolkotr , I checked the CI output and the changes in this MR are not related to these failing tests. As @fwinkl said, we should just wait for these tests to be fixed.

So, I am removing the "user-action-required" label.

removed review-user-action-required label

This merge request affects 1 package:

graphics/VP1/VP1Gui

This merge request affects 1 file:

graphics/VP1/VP1Gui/src/VP1EventDisplayEditor.cxx

Adding @rbianchi as watcher

added review-pending-level-1 label

resolved all threads

CI Result SUCCESS (hash 1b1e421f)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4882]