exstreme / Jcomments-4

Modified Jcomments 3.0.7 with Recaptcha 2 in comments. Adapted to Joomla 4. No compatibility with Joomla 1.5\2.5\3.x
16 stars 12 forks source link

Replace smilies with better versions, refresh comments list after new comment is saved #97

Closed iantrudell closed 5 months ago

iantrudell commented 2 years ago

Feel free to modify this (need to add update SQL I think for the smilies). The PNG versions are transparent and improve on the GIF versions.

For the refresh comments list part, the objective is to reload/refresh the comments list after a new comment is submitted/saved. Currently, it doesn't always update the comment lists and users frequently double-post as a consequence.

exstreme commented 2 years ago

1) May be better disable button? 2) How about delay with another message (or yoy set this value to 0 in settings?) 3) Can you describe, how to repeat this case?

iantrudell commented 1 year ago
  1. May be better disable button?
  2. How about delay with another message (or yoy set this value to 0 in settings?)
  3. Can you describe, how to repeat this case?
  1. Not sure what you mean by disable button
  2. Not sure what you mean
  3. Load a page with many comments, then submit a new comment. Currently, it won't reliably/automatically refresh the comments list to show the newly posted comment. With this change, it will always refresh the comments list after a user adds a new comment.
exstreme commented 1 year ago

@iantrudell may be can you share link, where we can reproduce and view the bug? And your branch has conflicts

Globulopolis commented 1 year ago

3. Load a page with many comments, then submit a new comment. Currently, it won't reliably/automatically refresh the comments list to show the newly posted comment. With this change, it will always refresh the comments list after a user adds a new comment.

I have seen this bug when comments displayed as list view(not a tree). List won't update(and no errors).

exstreme commented 5 months ago

Is it 18x18 too small resolution in 2024? And gif can have animation and can be transparent too! SQL isn't updatable, only for new installs