[#957] [BZ#1712873] Prevent migration task info popovers with long error messages from being pushed off the screen #962
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #957.
https://bugzilla.redhat.com/show_bug.cgi?id=1712873
The history of
.popover
max-width
continues...none
in [3579] Long string overflow resolved by removing popover max-width property #580.max-width: none
for all popovers to fix a similar issue: Improve per provider settings ui manageiq-ui-classic#2978, so that was still taking effect even without our override.max-width: none
here causes longer errors to run off the screen.I set the max-width of these specific task info popovers to a sensible 450px in this PR, which should be wide enough to prevent the first issue and narrow enough to prevent the second one.
Maybe we're done now?
This fix can be tested with the database called
long-task-error
from my collection, which has a plan "rhv-vddk" containing the long error shown in the screenshots below.Screens
Popover with the original

max-width: 220px
from bootstrap:Popover with

max-width: none
:Popover with

max-width: 450px
: