Skip to content

fix(gen2): [External] Fix warning icon height in Callout#3822

Open
alexchen-okta wants to merge 2 commits into
7.30from
ac-external-pr-3801
Open

fix(gen2): [External] Fix warning icon height in Callout#3822
alexchen-okta wants to merge 2 commits into
7.30from
ac-external-pr-3801

Conversation

@alexchen-okta

@alexchen-okta alexchen-okta commented Apr 16, 2025

Copy link
Copy Markdown
Contributor

Description:

Original PR from external contributor: #3801

PR Checklist

Issue:

Reviewers:

Screenshot/Video:

Downstream Monolith Build:

@alexchen-okta alexchen-okta changed the title fix(gen2): External contribution - Fix warning icon height in Callout fix(gen2): [External Fix warning icon height in Callout Apr 16, 2025
@alexchen-okta alexchen-okta changed the title fix(gen2): [External Fix warning icon height in Callout fix(gen2): [External] Fix warning icon height in Callout Apr 16, 2025
visibility: visible;
content: '';
height: 102%;
height: calc(100% + 2px);

@lesterchoi-okta lesterchoi-okta Apr 16, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://caniuse.com/calc

For IE11, can you check this against the "Known issues" tab?

@alexchen-okta alexchen-okta Apr 16, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, one of the known issues is that "IE11 is reported to not support calc() correctly in generated content"

@alexchen-okta

alexchen-okta commented Apr 16, 2025

Copy link
Copy Markdown
Contributor Author

Closing due to IE11 compatibility concerns. One of the known issues reported by https://caniuse.com/calc is "IE11 is reported to not support calc() correctly in generated content"

@alexchen-okta alexchen-okta reopened this Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants