Skip to content
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

[Bug]: FileIconType.documentsFolder icon is not registered #25646

Closed
2 tasks done
yume-chan opened this issue Nov 12, 2022 · 2 comments · Fixed by #25688
Closed
2 tasks done

[Bug]: FileIconType.documentsFolder icon is not registered #25646

yume-chan opened this issue Nov 12, 2022 · 2 comments · Fixed by #25688
Labels

Comments

@yume-chan
Copy link
Contributor

Library

React / v8 (@fluentui/react)

System Info

System:
    OS: Windows 10 10.0.25236
    CPU: (12) x64 AMD Ryzen 5 5600X 6-Core Processor
    Memory: 11.16 GB / 31.91 GB
  Browsers:
    Edge: Spartan (44.25236.1010.0), Chromium (107.0.1418.35), ChromiumDev (109.0.1481.0)
    Internet Explorer: 11.0.25236.1000

Are you reporting Accessibility issue?

no

Reproduction

https://codesandbox.io/s/suspicious-sunset-jswqxe?file=/src/icons.js

Bug Description

Actual Behavior

Use FileIconType.documentsFolder doesn't show anything, console prints this warning:

The icon "documentfolder64_1.5x_png" was used but not registered. See https://github.com/microsoft/fluentui/wiki/Using-icons for more information. 
    in IconBase (created by StyledIconBase)
    in StyledIconBase (created by StyledIconBase)
    in StyledIconBase (created by FileTypeIconBasicExample)
    in FileTypeIconBasicExample (created by App)
    in App 

Expected Behavior

It shows an icon

Logs

No response

Requested priority

Normal

Products/sites affected

No response

Are you willing to submit a PR to fix?

no

Validations

  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.
@github-actions github-actions bot added the Fluent UI react (v8) Issues about @fluentui/react (v8) label Nov 12, 2022
@ling1726
Copy link
Member

The bug is here:

const DOCUMENTS_FOLDER = 'documentfolder';

The correct name should be documentsfolder

@ling1726
Copy link
Member

@yume-chan we might not have capacity to get to this in the near future, would you be interested in submitting a fix, it should be fairly easy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
Development

Successfully merging a pull request may close this issue.

3 participants