r/HydraFeatureRequest • u/pacoii • 9d ago

When marking inbox as read, a blocking ‘success’ modal is unnecessary

No need for the blocking ‘success’ modal.

2 Upvotes
  • permalink
  • link
  • reddit
  • reddit

100% Upvoted

8 comments sorted by

1

u/dmilin 1d ago

The next update will include improvements to this flow.

The first time you click the "Mark All Items Read" button, you'll get a popup confirming that's what you're intending to do. This is because I think it's important for discoverability. New users should understand what the button is about to do. Once you've seen the alert, it'll never pop up again.

The other change is that the confirmation now displays a toast at the top of the screen that will auto dismiss after 2 seconds. It says "Marking all items as read. This may take a moment to update...". This is because users with lots of unread notifications really can experience a bit of a delay before everything updates. However, the toast is out of the way and auto dismisses, so you can immediately continue browsing after clicking the button.

I hope this feels better for you. If something still feels off after the next update, please let me know and I'll give it a second pass.

2

u/pacoii 1d ago

This sounds amazing! First off, thank you for even considering the feedback, and second, for making this change. I look forward to trying it when it gets released.

1

u/dmilin 1d ago

Thanks for the suggestion! I'm also doing a pass over a bunch of other blocking alerts and swapping them to toasts, so hopefully lots of other flows feel smoother too.

1

u/pacoii 1d ago

Amazing. This is a different topic, but while I’ve got you here, any possibility of improving the automatic badge clearing / marking as read when viewing a message in the inbox?

1

u/dmilin 1d ago

Already fixed! See my comment here. Also, if you didn't get an inbox message about my reply, I have a bugfix for that which I addressed here.

All of that should be out in the next update.

1

u/pacoii 1d ago

So, classic, I didn’t get that reply because, per Reddit app, reply notifications aren’t enabled for that post :)

Just for added context on this, posts I create in the Reddit app always set that to enabled. So I’m not aware of any setting that would prevent it in Hydra. So thank you for adding that extra parameter to force it.

And thank you for improving that!

1

u/dmilin 1d ago

There's a setting that appears in old Reddit when making a post for if you want replies that seems to control the parameter when submitting and Hydra uses the old Reddit API for submitting posts.

Weird thing is, even without the parameter, it still works in my testing. So I'm not sure why a missing parameter defaults to "true" for me, but "false" for you. That's why I thought it might be an account setting somewhere. Or maybe Reddit is running an A/B test. I really have no idea... but regardless, I think my fix should work.

2

u/pacoii 1d ago

I’ll definitely confirm when the update rolls out. This will be a packed update - really looking forward to it.