Skip to content

Rendered output displays "See also:" line twice #431

Open
@kbdharun

Description

@kbdharun

Expected behavior

Continuing tldr-pages/tldr#5190 (comment) (cc @sbrl).

The rendered output of pages should be displayed as is without a redundant "See also:" entry below the pages.

Actual behavior

Currently, the rendered output of pages displays the "See also:" line twice and in pages using a description like encfs it might be useful but still redundant.

image

Also in pages where we add it to an existing line in those cases this additional rendering doesn't take place:

image

What do you guys think, should we remove it? (IMO removing it will make the rendering consistent with other clients which doesn't do this)

If we decide to go with it, removing the below lines would resolve this issue:

if (page.seeAlso && page.seeAlso.length > 0) {
output.push('');
output.push('See also: ' + page.seeAlso.join(', '));
output.push('');
}

Environment

  • Operating system - Linux (Ubuntu LTS 22.04, Fedora Workstation 39), Windows (11)
  • Node.js version (node --version): v20.9.0
  • tldr-node-client version (tldr --version): 3.4.0

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions