Faster (and smaller) uploads in Discourse with Rust, WebAssembly and MozJPEG | Blog

Super psyched to see Discourse playing more with the power of Rust! :crab: :muscle:

imageflow is another impressive Rust lib in this space.

I took the liberty of sharing the news here:

14 Likes

This is something I have on my site right now? Meaning y’all shipped this level of improved functionality out to everyone just because you want a better product?

Part of me knows that’s true. I’ve been around enough to know how y’all roll, but damn. Part of me feels like I missed the part where this is an advanced optimization play.

7 Likes

Yes and yes. :smiley:

6 Likes

Freakin’ radical, man.

Bill Murray Thank You GIF by filmeditor

7 Likes

Getting lots of complaints from Android users about jumping cursors and degraded upload performance, unfortunately.

1 Like

I have a PR open that will greatly improve the cursor/placeholder handling during multiple file uploads

If all of the following are true:

  • You don’t care about storage costs
  • Most of your user base has good internet upload
  • Most of your audience browses from average or worse Android phones

You may want to either disable the feature or tweak the threshold for minimum size up a few megabytes.

8 Likes

That’s great to hear!

Sadly, many of my users have cheap-ish/older Android phones, so I won’t be re-enabling the feature any time soon. Haven’t had any problems myself on my iPhone 12.

3 Likes

Falco, would it be possible or of interest to provide a user setting for this, so people who want it can choose to use it? Or an option to “use classic image uploader” as craigslist does it? Selecting that button toggles the upload method.

3 Likes

Maybe tweak the minimum size for it, so it’s only triggers on file larger than 3MB, so old phones won’t reach it on their own camera pictures.

That was included in the original RFC but it was removed according to the comments received.

4 Likes

Hi Rafael,

I think we have a little problem with the new placeholder handling. It seems the images goes inline not a new line. Not just on composer pre optimization but regular uploads too. Thanks :slightly_smiling_face:

4 Likes

This should fix it

7 Likes

Hello Rafael,

Sorry I am late but tested just now. :confused: It works fine on multiple uploads if i drag images in new line but it seems the issue still there if drag image(s) the end of a text. Thank you :slightly_smiling_face:

3 Likes

Is it all good now @dodesz?

4 Likes

It seems to this one still happening.

3 Likes

Has this been fixed? @Falco

3 Likes

Sorry, what is a bug in that video? That looks exactly as I would expect it to work?

4 Likes

I missed this feature A setting to always insert images as separate paragraphs - #16 by andrei which is I think works well until this commit Faster (and smaller) uploads in Discourse with Rust, WebAssembly and MozJPEG | Blog - #9 by Falco not sure. :confused:

I have a video from my community about this feature when I presented to the members.

4 Likes

@sam can we address the above before the release please?

4 Likes

From what I can tell @andrei’s change was tightly scoped to the toolbar, increasing the scope so it also catches drag-and-drop and paste would be a bit risky for the release.

Also unsure if we want to do that?

8 Likes

5 posts were split to a new topic: Caret moves to end-of-line when uploading an image via toolbar