Skip to content

Commit

Permalink
Improve ordering
Browse files Browse the repository at this point in the history
  • Loading branch information
krlmlr committed Dec 23, 2023
1 parent 927b8ff commit fac5576
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
7 changes: 5 additions & 2 deletions R/rd-family.R
Original file line number Diff line number Diff line change
Expand Up @@ -51,12 +51,15 @@ topics_process_family <- function(topics, env) {
if (length(others) < 1)
next

by_file <- map_chr(aliases[others], function(x) {
other_aliases <- aliases[others]
other_aliases_order <- map_chr(other_aliases, function(x) escape(x[1]))

by_file <- map_chr(other_aliases[order(other_aliases_order)], function(x) {
obj <- find_object(x[1], env)
suffix <- if (is.function(obj$value)) "()" else ""
paste0("\\code{\\link{", escape(x[1]), "}", suffix, "}")
})
links <- paste(sort_c(by_file), collapse = ",\n")
links <- paste(by_file, collapse = ",\n")
seealso <- topics_process_family_prefix(family)

topic$add(rd_section("seealso", paste0(seealso, "\n", links)))
Expand Down
4 changes: 2 additions & 2 deletions tests/testthat/_snaps/rd-family.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
}
\seealso{
Other a:
\code{\link{foo2}()},
\code{\link{foo}()}
\code{\link{foo}()},
\code{\link{foo2}()}
}
\concept{a}

0 comments on commit fac5576

Please sign in to comment.