Skip to content

Commit

Permalink
Improve ordering
Browse files Browse the repository at this point in the history
  • Loading branch information
krlmlr committed Dec 25, 2023
1 parent 60621e8 commit 95a6610
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 7 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_c(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
8 changes: 7 additions & 1 deletion R/util-locale.R
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,10 @@ with_collate <- function(locale, code) {
force(code)
}

sort_c <- function(x, ...) with_collate("C", sort(x, ...))
sort_c <- function(x, ...) {
with_collate("C", sort(x, ...))
}

order_c <- function(x, ...) {
with_collate("C", order(x, ...))
}
8 changes: 4 additions & 4 deletions tests/testthat/_snaps/rd-family.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@
}
\seealso{
Other a:
\code{\link{foo2}()},
\code{\link{foo}()}
\code{\link{foo}()},
\code{\link{foo2}()}
}
\concept{a}
Expand All @@ -36,8 +36,8 @@
}
\seealso{
Other a:
\code{\link{foo1}()},
\code{\link{foo}()}
\code{\link{foo}()},
\code{\link{foo1}()}
}
\concept{a}
Expand Down

0 comments on commit 95a6610

Please sign in to comment.