Bug 1566565

Summary: Make all 'Alert' elements match Paternfly rules
Product: Red Hat Satellite Reporter: Andrii Balakhtar <abalakht>
Component: WebUIAssignee: sat6-team-ux <sat6-team-ux>
WebUI sub component: Katello QA Contact: vijsingh
Status: CLOSED WONTFIX Docs Contact:
Severity: high    
Priority: unspecified Keywords: Triaged
Version: Nightly   
Target Milestone: Unspecified   
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-06-03 12:05:41 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Andrii Balakhtar 2018-04-12 14:16:14 UTC
Description of problem:
It is expected that all WebUI Alerts (flash notification) elements should match next template:

<div class="alert alert-danger alert-dismissable">
  <button type="button" class="close" data-dismiss="alert" aria-hidden="true">
    <span class="pficon pficon-close"></span>
  </button>
  <span class="pficon pficon-error-circle-o"></span>
  <strong>Hey there is a problem!</strong> Yeah this is really messed up and you should <a href="#" class="alert-link">know about it</a>.
</div>

Alert text should have short summary in <strong> tag, and some details afterwards with no additional tags.

But instead there's no <strong> tag and entire message is placed inside 2 <span> tags:

<div class="alert toast-pf alert-success alert-dismissable">
    <button aria-hidden="true" type="button" class="close close-default">
        <span aria-hidden="true" class="pficon pficon-close"></span>
    </button>
    <span aria-hidden="true" class="pficon pficon-ok"></span>
    <span><span>Removed 1 host collections from activation key "test1".</span></span>
</div>

Such inconsistency forces us to write our own implementation of alert message for automation instead of reusing existing one in widgetastic_patternfly.


Version-Release number of selected component (if applicable):
6.4 upstream nightly

How reproducible:
Always

Steps to Reproduce:
1. Trigger any action which shows notification afterwards, e.g. add or remove host collection in activation key.
2. Check page source when notification pops up.
3.

Actual results:
Provided in description

Expected results:
Provided in description

Additional info:

Comment 2 Bryan Kearney 2019-05-02 19:06:54 UTC
The Satellite Team is attempting to provide an accurate backlog of bugzilla requests which we feel will be resolved in the next few releases. We do not believe this bugzilla will meet that criteria, and have plans to close it out in 1 month. This is not a reflection on the validity of the request, but a reflection of the many priorities for the product. If you have any concerns about this, feel free to contact Red Hat Technical Support or your account team. If we do not hear from you, we will close this bug out. Thank you.

Comment 3 Bryan Kearney 2019-06-03 12:05:41 UTC
Thank you for your interest in Satellite 6. We have evaluated this request, and while we recognize that it is a valid request, we do not expect this to be implemented in the product in the foreseeable future. This is due to other priorities for the product, and not a reflection on the request itself. We are therefore closing this out as WONTFIX. If you have any concerns about this, please do not reopen. Instead, feel free to contact Red Hat Technical Support. Thank you.