Fixing highlight width inconsistencies in list panel

[rashidakanchwala]

Description

Resolves #2002

Development notes

Known Issue: When hovering over the 'Expand Pipeline' icon, the entire width is not highlighted. This issue arises due to the complexity of the MUI Tree structure. Attempting to fix this causes other elements to break. A follow-up ticket will be created to address this bug.

Steph: Given its complexity, please let me know if this issue should be considered a low priority, as it does not affect the slicing issue.

QA notes

Checklist

• Read the contributing guidelines
• Opened this PR as a 'Draft Pull Request' if it is work-in-progress
• Updated the documentation to reflect the code changes
• Added new entries to the RELEASE.md file
• Added tests to cover my changes

[stephkaiser]

Thanks Rashida! some comments from me:

1. the full-width highlights looks great! it seems like parameters still have the shorter width, could we include parameters as well in this update?
2. selecting a child node and then closing the pipeline group doesnt bring the blue selection line to the parent level, can we also include this change? This would be useful in slicing when the contents of a pipeline are selected but not expanded

let me know if you want to discuss!

[Huongg]

Thanks Rashida! some comments from me:

1. the full-width highlights looks great! it seems like parameters still have the shorter width, could we include parameters as well in this update?
2. selecting a child node and then closing the pipeline group doesnt bring the blue selection line to the parent level, can we also include this change? This would be useful in slicing when the contents of a pipeline are selected but not expanded

let me know if you want to discuss!

hey @stephkaiser, please see my reply as below:

1. what you found there is actually a bug, when the parameter tag is off, user shouldn't be able to hover over it. I've fixed it now. So now if you turn the parameter tag on, hovering over the parameter should have the full width
2. For this one, let me try to adjust this in the slicing PR rather than here. Because if we do it here it will introduce a new behaviour, as in now if you select the node, close the pipeline, the node will be deselected anyway, so it will be hard to highlight the pipeline. However in the slicing we keep the list of the node so should be able to do this one in that PR

[stephkaiser]

hey @stephkaiser, please see my reply as below:

1. what you found there is actually a bug, when the parameter tag is off, user shouldn't be able to hover over it. I've fixed it now. So now if you turn the parameter tag on, hovering over the parameter should have the full width
2. For this one, let me try to adjust this in the slicing PR rather than here. Because if we do it here it will introduce a new behaviour, as in now if you select the node, close the pipeline, the node will be deselected anyway, so it will be hard to highlight the pipeline. However in the slicing we keep the list of the node so should be able to do this one in that PR

Thanks Huong!

1. I just had a look and this looks good to me!
2. I'll review this part in the slicing PR, thanks

[Huongg]

if so you're happy to approve the PR @stephkaiser so we can merge it?

[stephkaiser]

if so you're happy to approve the PR @stephkaiser so we can merge it?

done! thank you :)