Upload modal window with customisable warning / text area

andrei · July 2, 2021, 10:04am

The upload modal was removed in this PR:

FEATURE: use native file picker in composer (#13552)

committed 08:45AM - 30 Jun 21 UTC

We want to remove completely our custom modal for uploading files in composer an…d directly trigger the system file picker. This PR makes it happen. The fix is pretty simple since we already weren't using our custom modal on mobile. We just need to start using the same hidden <input type="file"> that we already use on mobile. It seems to be pretty tricky to test opening a system modal so I haven't added new tests. We already have other tests for file uploading though. We directly trigger jquery-File-Upload plugin hooks in those tests - https://github.com/discourse/discourse/blob/3dda926cb22751beced12b427dd14657b53b6dfe/app/assets/javascripts/discourse/tests/acceptance/composer-attachment-test.js#L89.

Now we just trigger system file pickers directly.

Topic		Replies	Views
Restrict uploads theme-component	16	3069	January 8, 2024
More flexible upload options for admin? feature	6	1044	May 30, 2019
Composer tip under specific tag topics theme-component	20	1170	February 21, 2023
Discourse Stickers theme-component	10	786	November 9, 2023
Provide a link to CommonMark documentation from the editor feature	16	1287	March 8, 2020

Upload modal window with customisable warning / text area

Related Topics