Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

Documentation for the recent mimetype files #1308

Merged
merged 1 commit into from
Jul 28, 2015
Merged

Conversation

rullzer
Copy link
Contributor

@rullzer rullzer commented Jul 10, 2015

PR for #1307

Not to much text. But then again how much is really needed?

CC: @MorrisJobke @PVince81 @Xenopathic

@rullzer rullzer added this to the 8.2-current milestone Jul 10, 2015

$ ./occ maintenance:mimetypesjs

This commands makes sure that the changes are propagate trough the system.
Copy link
Contributor

Choose a reason for hiding this comment

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

s/trough/through/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@MorrisJobke
Copy link
Contributor

Fine with me.

cc @carlaschroder Please have a look at the location. :)

@RobinMcCorkell
Copy link

We could do with a short list and description of common mimetype aliases in the relevant file, for example:

  • image
  • image/vector
  • audio
  • font
  • x-office/document
  • x-office/spreadsheet
  • x-office/presentation
  • text/code

Those are likely to be the most common ones that an administrator would want to add. And a note that states that to convert to an icon, the first part of the mimetype (the bit before the /) will be checked if the full mimetype cannot be found, so audio/foobar will still get an audio icon.

@carlaschroder
Copy link
Contributor

This is good, @rullzer, thank you. I agree with Xenopathic; some examples and a little more explanation is very helpful.
@MorrisJobke I'm going to do a bit of re-organizing, so this is fine where it is for now.

@carlaschroder
Copy link
Contributor

I'll go ahead and merge this; it can always be expanded later. (meaningful look at @Xenopathic)

carlaschroder pushed a commit that referenced this pull request Jul 28, 2015
Documentation for the recent mimetype files
@carlaschroder carlaschroder merged commit 1924b1a into master Jul 28, 2015
@carlaschroder carlaschroder deleted the mimetypes branch July 28, 2015 16:00
@RobinMcCorkell
Copy link

As a wise man once said, "Ask and thou shalt receive": #1373 😄

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

Successfully merging this pull request may close these issues.

4 participants