Skip to content

Task/fp 228 new icons#131

Closed
wesleyboar wants to merge 9 commits intomasterfrom
task/FP-228-new-icons
Closed

Task/fp 228 new icons#131
wesleyboar wants to merge 9 commits intomasterfrom
task/FP-228-new-icons

Conversation

@wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Aug 3, 2020

Closed

In favor of #132. The two other sub-tasks that were gonna be done have been moved to Safehold.

Overview:

Remaining work from sub-tasks of FP-228 (which might instead have smaller independent PRs).

PR Status:

  • Work in Progress. (pending creation and merge of sub-task PRs)

Related Jira tickets:

Summary of Changes:

(pending)

Testing Steps:

  1. (pending)

UI Photos:

(pending)

Notes:

(pending)

wesleyboar and others added 7 commits July 13, 2020 19:33
* FP-515: Add Icon component from FP-354

* FP-515: Add Icon component usage from FP-354

* FP-515: Install Cortal 1.2

- Disable linting of `icon.fonts.css`.
- Support WOFF in Webpack config.
- Update several files (and tests) to use new icon names:
    - AppIcon
    - AppBrowser
    - DataFilesListingCell
    - DataFilesSidebar
    - DataFilesToolbar
    - Sidebar
    - TicketModal
- Replace old Core Portal font with Cortal Icons font.
- Migrate font styles to their own stylesheet.
- Add icon aliases:
    - icon-collapse (icon-contract)

* FP-515: Disable an "extra icon"
@codecov
Copy link

codecov bot commented Aug 3, 2020

Codecov Report

Merging #131 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #131   +/-   ##
=======================================
  Coverage   55.11%   55.11%           
=======================================
  Files         222      222           
  Lines        7786     7786           
  Branches     1084     1084           
=======================================
  Hits         4291     4291           
  Misses       3291     3291           
  Partials      204      204           
Flag Coverage Δ
#javascript 54.83% <ø> (ø)
#unittests 55.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...c/components/Applications/AppBrowser/AppBrowser.js 3.84% <ø> (ø)
client/src/components/_common/Icon/Icon.js 100.00% <ø> (ø)

@wesleyboar
Copy link
Member Author

See #132, instead.

@wesleyboar wesleyboar closed this Aug 6, 2020
@wesleyboar wesleyboar deleted the task/FP-228-new-icons branch August 7, 2020 15:33
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.

1 participant