Skip to content

Conversation

@annda
Copy link
Contributor

@annda annda commented Jul 21, 2020

Usability improvement: displays mime types which are whitelisted in the configuration.

@splitbrain
Copy link
Collaborator

I just gave it a try. Works fine and is a useful improvement. However, I'm not so happy with the UI. I would prefer this to

a) take up less space (maybe move it next to the browse button?)
b) not use the jQuery UI accordion but a simple link that appends the list of extensions in the same paragraph.

screenshot-localhost-2020 07 22-11_27_22

@annda
Copy link
Contributor Author

annda commented Jul 22, 2020

What about putting it next to the upload limit? The upload form is built completely in JavaScript, so to access the mime types, they would have to be part of JSINFO, which is probably unnecessary.

@splitbrain
Copy link
Collaborator

Both are limitations of what you can upload, so yes, it's fine to have them together.

@annda annda force-pushed the media-extensions branch from 157392f to 56fc6b1 Compare July 22, 2020 11:16
@annda
Copy link
Contributor Author

annda commented Jul 22, 2020

The UI is much more compact now.

The recent Travis failure seems to be unrelated and random. @splitbrain could you please re-trigger the build?

@splitbrain splitbrain merged commit 56fc6b1 into dokuwiki:master Jul 23, 2020
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.

2 participants