Enterprise: Notifications: No background color in Internet Explorer

Created on 27 Mar 2019  路  8Comments  路  Source: infor-design/enterprise

Describe the bug
Notification: No background color in Windows Internet Explorer.

To Reproduce
Steps to reproduce the behavior:

  1. Open Internet Explorer, go to https://4170-rc0-enterprise.demo.design.infor.com/components/notification/example-index.html#
  2. See issue: No background color on the Notifications

Expected behavior
There should be a background color in Notifications

Version

  • ids-enterprise: v4.11.0 - v4.17.0-rc0

Screenshots
Screen Shot 2019-03-27 at 6 06 00 PM

Platform

  • Windows Internet Explorer

Additional context
No Additional Context.

[1] type

All 8 comments

This relates to https://github.com/infor-design/design-system/issues/294 once we change this colors on a new issue when thats decided we should also test and fix this for IE with the new colors. This bug occurred when this component was created in 4.14 and was'nt noticed until now.

Just curious - do we know why the current colors (background-color: rgba(213, 246, 192, 0.4) for example) aren't showing on IE? Is it because of the opacity? If that's the case, going "solid" in the proposed solution would fix this bug.

@kayiuho I think its just because we use css RGBA there which is not supported on IE. We should convert these to HEX or use the HEX OR RGB with an actually css opacity. So, coding error.

We still need to make the changes here

Further debugging:

  • So the real issue is that the classes are not being set on the notification elements:
    Screen Shot 2019-04-22 at 11 58 24 AM
  • Which happens because of this in the JS code:
    Screen Shot 2019-04-22 at 12 02 32 PM

  • Because we do this in the JS:

    this.notificationEl.classList.add('notification', this.settings.type);
    
Was this page helpful?
0 / 5 - 0 ratings

Related issues

tmcconechy picture tmcconechy  路  16Comments

jamie-norman picture jamie-norman  路  19Comments

jamie-norman picture jamie-norman  路  18Comments

marclouisgenedeguzman picture marclouisgenedeguzman  路  15Comments

Sovia picture Sovia  路  16Comments