Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(core/cluster): Clicking an instance breaks rendering of cluster pods #8866

Merged
merged 2 commits into from
Feb 1, 2021

Conversation

caseyhebebrand
Copy link
Contributor

The AutoSizer was only listening to a window resize, but when an instance or server group is selected, rows also need to be resized. This caused the server group views to render on top of each other once the details panel was opened.

This PR adds a subscription to listen for opening/closing the details panel so that the cluster pods can re-render with appropriate heights.

Copy link
Contributor

@alanmquach alanmquach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch

@caseyhebebrand caseyhebebrand merged commit c7a58cd into spinnaker:master Feb 1, 2021
@caseyhebebrand caseyhebebrand deleted the cluster-pod-render branch February 1, 2021 19:19
christopherthielen added a commit that referenced this pull request Feb 3, 2021
274944f feat(md): Collasible metadata elements  (#8880)
c7a58cd fix(core/cluster): Clicking an instance breaks rendering of cluster pods (#8866)
76cd8cd feat(core): Add a new variable that is interpolated in instance links (#8868)
b3ae753 fix(core): add vertical padding on page owner nav link (#8876)
7d94b69 feat(core): Expose raw subnet ID (#8877)
mergify bot pushed a commit that referenced this pull request Feb 3, 2021
274944f feat(md): Collasible metadata elements  (#8880)
c7a58cd fix(core/cluster): Clicking an instance breaks rendering of cluster pods (#8866)
76cd8cd feat(core): Add a new variable that is interpolated in instance links (#8868)
b3ae753 fix(core): add vertical padding on page owner nav link (#8876)
7d94b69 feat(core): Expose raw subnet ID (#8877)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants