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

Changed mapping to font #34419

Merged
merged 2 commits into from
Aug 3, 2023
Merged

Changed mapping to font #34419

merged 2 commits into from
Aug 3, 2023

Conversation

dos-2
Copy link
Contributor

@dos-2 dos-2 commented Oct 4, 2022

old
Previous code had "application/font-woff": "image"
which caused the error.
new
New code has fixed the mapping.

@szaimen szaimen added the 3. to review Waiting for reviews label Oct 4, 2022
@szaimen szaimen added this to the Nextcloud 26 milestone Oct 4, 2022
@tcitworld tcitworld mentioned this pull request Oct 4, 2022
Copy link
Member

@juliushaertl juliushaertl left a comment

Choose a reason for hiding this comment

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

Thanks for your pull request.

This should also get adjusted in core/js/mimetypelist.js

In addition it might make sense for other font related mime types as well

@juliushaertl
Copy link
Member

@jancborchardt Maybe you can come up with a nice icon for font files?

@jancborchardt
Copy link
Member

Yes, format-font would be a good icon. (Not in Material Design core apparently, but from https://mdisearch.com/ )

@blizzz blizzz mentioned this pull request Feb 1, 2023
@skjnldsv skjnldsv mentioned this pull request Feb 23, 2023
@blizzz blizzz mentioned this pull request Mar 7, 2023
@blizzz blizzz modified the milestones: Nextcloud 26, Nextcloud 27 Mar 9, 2023
This was referenced May 3, 2023
@blizzz blizzz mentioned this pull request May 17, 2023
@blizzz blizzz modified the milestones: Nextcloud 27, Nextcloud 28 May 23, 2023
dos-2 and others added 2 commits August 3, 2023 04:53
* Updated mimetypelist icon mapping

Signed-off-by: Dany1718 <daniel.dossantos@ufl.edu>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
Copy link
Contributor

susnux commented Aug 3, 2023

I rebased the PR and pushed a commit providing the icon @jancborchardt suggested, it looks like this now:

image

@susnux susnux requested a review from Fenn-CS August 3, 2023 03:16
@susnux susnux merged commit 9a7e2b1 into nextcloud:master Aug 3, 2023
34 of 37 checks passed
@welcome
Copy link

welcome bot commented Aug 3, 2023

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@kesselb
Copy link
Contributor

kesselb commented Aug 3, 2023

The failing CheckerTest looks related

Screenshot from 2023-08-03 21-55-13

@kesselb
Copy link
Contributor

kesselb commented Aug 3, 2023

Fix: #39705

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

Successfully merging this pull request may close these issues.

Icons for font files are incorrect in Files app
8 participants