Veritula – Meta

Showing only #1986 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 about 1 month ago·#1986
2nd of 5 versions

Bug: when cycling through ‘filtered’ revisions (meaning there are more revisions that don’t lead to the highlighted idea), the criticism badge can change count for the same revision.

CriticismCriticized2Archived
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 4 days ago·#2967
2nd of 2 versions

That in and of itself isn’t a bug if the different revisions leading to the highlighted idea have different numbers of criticisms.

The issue also isn’t that there are other revisions that don’t lead to the highlighted idea.

The real issue is twofold:

  1. On page render, no matter how many children/criticisms a parent of the highlighted idea has, only the highlighted idea is shown – that’s fine so far – but the displayed criticism count may be higher if there are criticisms that are not being shown. So there’s a mismatch.
  2. As you cycle through the parent’s versions, the children are suddenly not being filtered anymore, and the highlighted idea suddenly has siblings. Now the criticism count on the parent does always match the number of shown criticisms, but it seems arbitrary to suddenly not filter the children anymore.
Criticism of #1986
Dennis Hackethal’s avatar

Superseded by #2965. This comment was generated automatically.

Criticism of #1986