Kind of on the fence on this one, because humans can be awful and deserve to be criticized by future generations just like we do when see how bad the past was. The problem comes with the preachiness with a little hypocrisy and smugness appears.
Idk, maybe they deserve to be a little smug.
On a second look, you get the impression that Riker was criticizing their very quality as humans. As in the way they dress, the clothes they wore, the way they spoke, being from such a primitive era with odd customs. As if the 24th century had a much higher quality of human period, due to evolution.
The show itself was hinting at it when it implied TV watching was an outdated thing 'primitive humans did. Or commenting on some other behaviors from this era.
Crusher seemed to criticize their choices, but a few episodes later they meet a 24th century human that wanted to stay alive so bad, he transferred his mind into Data's.
One thing I do wonder is, why then do they immerse themselves so much in the early and mid 20th century when they use the holodeck? Picard, Riker, Crusher, Laforge, all have willingly done it.
.