graphql-relay-js icon indicating copy to clipboard operation
graphql-relay-js copied to clipboard

[RFC] Backwards arguments are equivalent to forward arguments

Open nijynot opened this issue 7 years ago • 0 comments

I wrote a little about it here, but basically after = Math.max(before - last - 1, 0).

So if you implement first and after, you automatically get the functionality of last and before without needing to change much at all as first and after is the same as last and before. It makes me think that the equivalency should be in the spec and that we should add the function below to graphql-relay-js.

export function transformBackwardToForward(args) {
  const { last, before } = args;
  const afterOffset = Math.max(cursorToOffset(before) - last - 1, 0);

  return { first: last, after: offsetToCursor(afterOffset) };
}

Has anyone else solved pagination this way? This might be worth looking into as it seems that many have problems with pagination by looking at #94. What do you guys think about this implementation overall, good/bad?

nijynot avatar Feb 18 '18 18:02 nijynot