-
Notifications
You must be signed in to change notification settings - Fork 615
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CONSOLE-3031: Remove font-awesome
and openshift-logos-icon
#14603
base: master
Are you sure you want to change the base?
Conversation
@logonoff: This pull request references CONSOLE-3031 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
80ef183
to
a7180c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments around imports and icons that are available in PatternFly
...end/packages/console-shared/src/components/markdown-extensions/inline-clipboard-extension.ts
Show resolved
Hide resolved
frontend/packages/console-shared/src/components/markdown-extensions/inline-execute-extension.ts
Show resolved
Hide resolved
frontend/packages/console-shared/src/components/markdown-extensions/inline-execute-extension.ts
Show resolved
Hide resolved
frontend/packages/console-shared/src/components/markdown-extensions/showdown-extension.scss
Show resolved
Hide resolved
frontend/packages/console-shared/src/components/markdown-extensions/utils.ts
Outdated
Show resolved
Hide resolved
a38fc80
to
ea842eb
Compare
@logonoff: This pull request references CONSOLE-3031 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
/retest |
@logonoff: This pull request references CONSOLE-3031 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
ea842eb
to
316efd8
Compare
adding labels because this is debt: /label px-approved QE review: /cc @yapei |
@logonoff: This pull request references CONSOLE-3031 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
@logonoff: This pull request references CONSOLE-3031 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
3f39c59
to
7947d02
Compare
/approve |
/label tide/merge-method-squash |
7947d02
to
ba9c212
Compare
Also remove `openshift-logos-icon`
ba9c212
to
1c8bac1
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: logonoff, rhamilto The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test okd-scos-e2e-aws-ovn |
1 similar comment
/test okd-scos-e2e-aws-ovn |
@logonoff: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Checked on cluster launched against the pr, check some pages on console including the screenshots in the pr description, there is not regression issue found. |
@logonoff: This pull request references CONSOLE-3031 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
CONSOLE Features and Fixes:
https://issues.redhat.com/browse/CONSOLE-3031
Solution description
font-awesome
openshift-logos-icon
due to non-usagefa-spin
was changed to a customco-spin
,co-external-link
andco-goto-arrow
were replaced with mask-images (so that the icons can retain the text color while keeping usage the same),co-modal-btn-link
was removed due to non-usage across actively maintained dynamic plugins on GitHubgetSvgFromPfIconConfig
was written to generate an SVG element from the PatternFly icon exportsTest cases:
Screen shots / gifs / design review:
Corporate needs you to find the differences between this picture and this picture:
Before:
After: