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

webadmin: fix alignment on Resource Allocation popup #495

Merged
merged 1 commit into from
Jul 4, 2022

Conversation

sleviim
Copy link
Member

@sleviim sleviim commented Jun 26, 2022

Screenshot: https://imgur.com/a/w1dMuCU

This patch fixes the alignment under Resource Allocation > Disk
Allocation, when cloning a VM.
The disk panel is being first created as part of the view initiation.
Determining the DC quota level (disabled/enforced) affects the number of
columns appear in the disk allocation view.
Since updating the quota happens later (not as part of initialization),
The columns' width can't be changed unless re-creating the disks lists
in the view, so the view needs to be created with the updated width.

Bug-Url: https://bugzilla.redhat.com/2081546
Signed-off-by: Shani Leviim sleviim@redhat.com

@sleviim sleviim force-pushed the bug-2081546 branch 2 times, most recently from dd06280 to 7edaa2e Compare June 28, 2022 14:55
@sleviim sleviim requested a review from ahadas June 28, 2022 14:56
Screenshot: https://imgur.com/a/CAIAnI1

This patch fixes the alignment under Resource Allocation > Disk
Allocation, when cloning a VM.
The disks' panel is being first created as part of the view initiation.
Determining the DC quota level (disabled/enforced) effects the number of
columns appear in the disk allocation view.
Since updating the quota happens later (not as part of initialization),
The columns' width can't be changed unless re-creating the disks lists
in the view, so the view needs to be created with the updated width.

Bug-Url: https://bugzilla.redhat.com/2081546
Signed-off-by: Shani Leviim <sleviim@redhat.com>
@ahadas
Copy link
Member

ahadas commented Jun 30, 2022

/ost

1 similar comment
@ahadas
Copy link
Member

ahadas commented Jul 1, 2022

/ost

@ahadas ahadas merged commit 50e0bb9 into oVirt:master Jul 4, 2022
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.

None yet

2 participants