this post was submitted on 25 Aug 2023
184 points (98.9% liked)

Sync for Lemmy

15131 readers
14 users here now

๐Ÿ‘€


Welcome to Sync for Lemmy!

Download Sync for Lemmy


Welcome to the official Sync for Lemmy community.

The rules for posting and commenting, besides the rules defined here for lemmy.world, are as follows:

Community Rules


1- No advertising or spam.

All types of advertising and spam are restricted in this community.



Community Credits

Artwork and community banner by: @[email protected]


founded 1 year ago
MODERATORS
 

Key points

  • Fixed most comment loading / display issues

New

  • Added an option to specify the initial comment chain depth loading (default 3)

Github issues closed

  • View more button not working properly #132
  • Terrible performance on a post with a ton of images #328
  • Sharing to Sync link handler gives a [TODO deeplink] toast message #149
  • Image in comments not appearing #306
  • Comments hierarchy messed up #241
  • Comment section not displaying properly when sorted by Top #133
  • View More button appears in the wrong place and behaves wrongly #233

Other fixes

  • Fixed an issue where disabling recording history caused a crash
  • Fixed another view more issue
  • Added the toast back in for view more
  • Fixed an issue with deep links not opening correctly
  • Reverted the behavior of clicking on a comment from messaging, more context is now shown
  • Reverted the behavior of clicking on an image in comments loading all as a gallery (buggy)
  • Resolving object requests are now cached
you are viewing a single comment's thread
view the rest of the comments
[โ€“] [email protected] 5 points 1 year ago (1 children)

They seem to be snackbars and should be

[โ€“] [email protected] 5 points 1 year ago* (last edited 1 year ago) (1 children)

Nah, they're toasts. At least the view more ones.

They stop working when I disable notifications, you can also tell by minimizing the app while they show. If they stay on the screen after its minimized they are toasts.

That's why they are tied to notifications, because they can show outside the app, unlike things like snackbars

[โ€“] [email protected] 4 points 1 year ago

Ah, I didn't check throughly enough. Thanks

They then should be snackbars instead