Veritula – Meta

Showing only #3002 and its comments.

See full discussion
  Log in or sign up to participate in this discussion.
With an account, you can revise, criticize, and comment on ideas.

Discussions can branch out indefinitely. Zoom out for the bird’s-eye view.
Dennis Hackethal’s avatar
Dennis HackethalOP revised 2 days ago·#3002
5th of 5 versions

The displayed criticism count for a filtered parent can differ from the number of displayed criticisms.

CriticismCriticized1Archived
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#2008
3rd of 3 versions

Any filtered idea should always display only the count of shown criticisms.

Criticized1
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#2001
2nd of 2 versions

That could mislead people into thinking a revision has no pending criticisms, which would be bad for error correction.

Criticism of #2008Criticized1
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 month ago·#1992

The instructions at the top of the page are clear that not all ideas are being rendered.

Criticism of #2001
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 month ago·#1999

People could easily miss or forget that.

Criticism of #1992Criticized1
Dennis Hackethal’s avatar

For filtered parents, I could put an asterisk behind the count. On hover, explain that the total number of pending criticisms may be greater on the unfiltered view.

Criticism of #1999
Dennis Hackethal’s avatar

As with #2098, implementing an accurate count of the number of shown criticisms gets very tricky once the user starts submitting new criticisms on filtered parents.

Criticism of #2008
Dennis Hackethal’s avatar
Dennis HackethalOP revised 1 day ago·#3014
2nd of 2 versions

For all ideas, the total number of pending criticisms (if any) should always be shown, even if they are not all being rendered. For filtered parents, I could put an asterisk behind the count. On hover, explain that some pending criticisms may be hidden due to filtering.

Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#2098
4th of 4 versions

Any filtered ideas should show a criticism label displaying n / m for the count, where n is the number of rendered criticisms and m is the number of total criticisms.

An explanation could accompany the n / m display, like a title on hover.

That way, there should never be any confusion as to a mismatch between the total vs rendered number of pending criticisms.

In addition, when looking at a deeply nested idea on ideas#show and submitting a criticism on a parent, I need to make sure the updated badges take into account that newly submitted criticism, even though the new criticism would not show after refreshing the page.

Criticized1
Dennis Hackethal’s avatar

I have this working to the point that it shows n / m, but getting the counter to update properly when new criticisms are posted on filtered parents is surprisingly difficult – so difficult the juice may not be worth the squeeze.

Criticism of #2098
Dennis Hackethal’s avatar
Dennis HackethalOP revised 2 days ago·#2999
2nd of 2 versions

For any filtered parent, the criticism badge could be shown without a count.

Criticized1
Dennis Hackethal’s avatar

But users are expecting a count.

Criticism of #2999
Dennis Hackethal’s avatar
Dennis HackethalOP revised 2 days ago·#3005
2nd of 2 versions

I could get rid of the count everywhere, even on unfiltered views. That would have the added benefit that users wouldn’t prefer one problematic idea over another just because it has fewer pending criticisms.

Criticism of #3001Criticized1
Dennis Hackethal’s avatar

Still, the count is valuable in that it shows how many criticisms need to be addressed to restore an idea.

Criticism of #3005
Dennis Hackethal’s avatar

#3014 fixes this. Implemented as of c3247d5.

Criticism of #3002