You are viewing a single comment's thread from:

RE: Steem Curation Extension: New version available with more info for curators

in Steem Dev14 days ago

I think it's fixed.

  1. For each resteemed post, it now checks if the feed owner follows the author or if the feed owner is the author. If either of those checks is true, the post won't be hidden. I save the following status for 30 minutes in order to minimize web traffic to the Steem API.
  2. While testing that I also noticed that if I scroll down too far in the feed, performance gets really bad, so I put in a limit at 200 posts. After that, the checkbox to toggle resteems disappears.

The second problem was actually pretty bad, so I went ahead and merged it into the main branch.

Sort:  

Looks good, I did some tests on my feed and all the resteems from followed accounts remained visible. The felt page load speed is the same as before. Wonderful, thanks for the fix!

 13 days ago 

While testing that I also noticed that if I scroll down too far in the feed, performance gets really bad, so I put in a limit at 200 posts. After that, the checkbox to toggle resteems disappears.

Why would performance decrease as the number of posts increases? Does the plugin re-evaluate every post at each additional load? Or is it parsing the generated HTML each time?

 12 days ago 

Why would performance decrease as the number of posts increases? Does the plugin re-evaluate every post at each additional load? Or is it parsing the generated HTML each time?

It's only when you check/uncheck the box, so I'm not sure if it's really avoidable. In order to hide/show resteems, it has to move all the other posts up or down. When I first encountered the problem, I was on the verge of crashing my browser when I tried to toggle "show resteems" on. Even 200 posts takes 2-3 seconds to rearrange things with my configuration. Tolerable, but slower than I'd like.

Coin Marketplace

STEEM 0.24
TRX 0.21
JST 0.036
BTC 97455.52
ETH 3338.22
USDT 1.00
SBD 3.34