openstack/octavia-dashboard
Code Issues Proposed changes

Fix popover attributes in member add dialog

Browse Source 
Using popovers in the member table is broken in chrome 114, because of
their new "popover" feature.

Closes-Bug: #2024188
Change-Id: Idb009295495c1d8c732e792a72d58912e0e8d3a7
(cherry picked from commit 9cc4bfac58)
(cherry picked from commit 6f990e09e7)
(cherry picked from commit bd1a329b46)
(cherry picked from commit b7785d206e)
(cherry picked from commit 3eae62925e)
This commit is contained in:
Gregory Thiemonge 2023-06-16 08:22:10 -04:00
parent 994011a2e4
commit 2bca0ed072
2 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
octavia_dashboard/static/dashboard/project/lbaasv2/workflow/members/members.html
View File

@ -54,7 +54,7 @@
ng-required="true" ng-disabled="row.allocatedMember"
popover-placement="top" popover-append-to-body="true"
popover-trigger="focus"
ng-attr-popover="{$ memberDetailsForm[row.id + '-address'].$invalid && memberDetailsForm[row.id + '-address'].$dirty ? ctrl.ipError : '' $}">
uib-popover="{$ memberDetailsForm[row.id + '-address'].$invalid && memberDetailsForm[row.id + '-address'].$dirty ? ctrl.ipError : '' $}">
</div>
<span ng-if="row.addresses.length === 1">{$ row.address.ip $}</span>
<div ng-if="row.addresses.length > 1"
@ -83,7 +83,7 @@
ng-required="true" ng-disabled="row.allocatedMember"
popover-placement="top" popover-append-to-body="true"
popover-trigger="focus"
ng-attr-popover="{$ memberDetailsForm[row.id + '-port'].$invalid && memberDetailsForm[row.id + '-port'].$dirty ? ctrl.portError : '' $}">
uib-popover="{$ memberDetailsForm[row.id + '-port'].$invalid && memberDetailsForm[row.id + '-port'].$dirty ? ctrl.portError : '' $}">
</div>
</td>
<td class="rsp-p1">
@ -94,7 +94,7 @@
ng-disabled="row.allocatedMember"
popover-placement="top" popover-append-to-body="true"
popover-trigger="focus"
ng-attr-popover="{$ memberDetailsForm[row.id + '-weight'].$invalid && memberDetailsForm[row.id + '-weight'].$dirty ? ctrl.weightError : '' $}">
uib-popover="{$ memberDetailsForm[row.id + '-weight'].$invalid && memberDetailsForm[row.id + '-weight'].$dirty ? ctrl.weightError : '' $}">
</div>
</td>
<td class="actions_column">
@ -126,7 +126,7 @@
ng-disabled="row.allocatedMember"
popover-placement="top" popover-append-to-body="true"
popover-trigger="focus"
ng-attr-popover="{$ memberDetailsForm[row.id + '-monitor-address'].$invalid && memberDetailsForm[row.id + '-monitor-address'].$dirty ? ctrl.ipError : '' $}">
uib-popover="{$ memberDetailsForm[row.id + '-monitor-address'].$invalid && memberDetailsForm[row.id + '-monitor-address'].$dirty ? ctrl.ipError : '' $}">
</dd>
</dl>
<dl class="col-lg-5 col-md-5 col-sm-5">
@ -138,7 +138,7 @@
ng-disabled="row.allocatedMember"
popover-placement="top" popover-append-to-body="true"
popover-trigger="focus"
ng-attr-popover="{$ memberDetailsForm[row.id + '-monitor-port'].$invalid && memberDetailsForm[row.id + '-monitor-port'].$dirty ? ctrl.portError : '' $}">
uib-popover="{$ memberDetailsForm[row.id + '-monitor-port'].$invalid && memberDetailsForm[row.id + '-monitor-port'].$dirty ? ctrl.portError : '' $}">
</dd>
</dl>
<dl class="col-lg-5 col-md-5 col-sm-5">

6
releasenotes/notes/fix-chrome-member-dialog-154981a7794d5fb4.yaml Normal file
View File

@ -0,0 +1,6 @@
---
fixes:
- |
Fixed a bug when displaying the member list, some fields were not displayed
correctly and new members could not be added.
This bug affected only Google Chrome 114 and above.