Veritula – Meta

Showing only #2965 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 1 day ago·#2965

Bugs when cycling through ‘filtered’ revisions:

  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.
CriticismCriticized22 pending criticisms
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#2008

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

Criticized11 pending criticism
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#2001

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

Criticism of #2008
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 #2001Criticized11 pending criticism
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 month ago·#1999

People could easily miss or forget that.

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

For all ideas, the total number of pending criticisms (if any) should always be shown, even if they are not all being rendered.

Criticized11 pending criticism
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#1995

If no criticisms are being displayed, yet the label says an idea has n pending criticisms, that might confuse people. More generally, any mismatch between rendered vs counted criticisms could confuse people.

Criticism of #1993
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 month ago·#1997

See #1992: “The instructions at the top of the page are clear that not all ideas are being rendered.”

Criticism of #1995Criticized11 pending criticism
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 month ago·#2000

See #1999: “People could easily miss or forget that.”

Criticism of #1997
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 month ago·#2098

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.

Criticized11 pending criticism
Dennis Hackethal’s avatar
Dennis HackethalOP, about 6 hours ago·#2997

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

Shouldn’t have more than one criticism at a time.

Criticism of #2965
Dennis Hackethal’s avatar

Superseded by #2970. This comment was generated automatically.

Criticism of #2965