Skip to content

Icons not displayed in FilePicker when using german language #1978

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

Open
geoo opened this issue Apr 22, 2025 · 3 comments
Open

Icons not displayed in FilePicker when using german language #1978

geoo opened this issue Apr 22, 2025 · 3 comments

Comments

@geoo
Copy link

geoo commented Apr 22, 2025

Category

[ ] Enhancement

[x] Bug

[ ] Question

Version

Please specify what version of the library you are using: [3.20.0]

Expected / Desired Behavior / Question

Icons are displayed when using the FilePicker component

Observed Behavior

When using the compact or list view of the FilePicker component the icons are not displayed if using german language (and a lot other languages)

Image

The "FolderIconUrl" in https://github.com/pnp/sp-dev-fx-controls-react/blob/master/src/loc/de-de.ts is pointing to a non existing svg ("https://spoprod-a.akamaihd.net/files/odsp-next-prod_2019-01-11_20190116.001/odsp-media/images/itemtypes/20/folder.svg")

Thanks!

Copy link

Thank you for submitting your first issue to this project.

@Ateina
Copy link
Contributor

Ateina commented Apr 23, 2025

Hi @michaelmaillot @joelfmrodrigues!

So, I suppose, correct link for FolderIconUrl would be folder icon link

After a quick investigation, I noticed several other icons referencing non-existent paths, but they are wrong for all languages:

  • OneDriveEmptyFolderIconUrl, maybe could be replaced with Font Icon

  • ODPhotoIconUrl, possible replacement photo icon link
  • PhotoIconUrl - only used in a commented block of code, maybe could be removed completely

@michaelmaillot
Copy link
Collaborator

Hi @geoo & @Ateina, thanks for raising this and the investigation!

Looks like "system" icons are refered to ODSP CDN, which location evolves with released versions and I'm not sure this should be handled like this in a long term perspective.

@AJIXuMuK do you know if there's a better way to get those generic icons, instead of using Akamai URLs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants