Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Site: show top-level option next to flags on command line reference #18065

Open
jschaf opened this issue Apr 12, 2023 · 2 comments
Open

Site: show top-level option next to flags on command line reference #18065

jschaf opened this issue Apr 12, 2023 · 2 comments
Labels
P2 We'll consider working on this in future. (Assignee optional) team-Documentation Documentation improvements that cannot be directly linked to other team labels team-Remote-Exec Issues and PRs for the Execution (Remote) team type: documentation (cleanup)

Comments

@jschaf
Copy link

jschaf commented Apr 12, 2023

Page link:

https://bazel.build/reference/command-line-reference

Problem description (include actual vs expected text, if applicable):

When navigating to an anchor link, figuring out which top-level option (common, build, query, etc.) the flag belongs to is difficult. For example, when navigating to https://docs.bazel.build/versions/main/command-line-reference.html#flag--experimental_remote_build_event_upload, the only way to figure out the top-level option is to scroll up. Since the page is massive, it's easy to miss the top-level option.

The top-level option is important because I need to know it to update .bazelrc. For example, buildbuddy-io/buildbuddy#3731

Where do you see this issue? (include link to specific section of the page, if applicable)

https://docs.bazel.build/versions/main/command-line-reference.html#flag--experimental_remote_build_event_upload

Any other information you'd like to share?

Some common patterns for improving positioning information for large pages like this:

  1. Show the top-level option as a fixed header to the left when scrolling.
  2. Show the top-level option on every flag.
  3. Highlight the top-level option when scrolling.
@jschaf jschaf added team-Documentation Documentation improvements that cannot be directly linked to other team labels type: documentation (cleanup) untriaged labels Apr 12, 2023
@ShreeM01 ShreeM01 removed their assignment Apr 12, 2023
@keertk keertk added P2 We'll consider working on this in future. (Assignee optional) and removed untriaged labels Jun 12, 2023
@ian-h-chamberlain
Copy link

See also #3758 which proposes a different solution (but I think would solve the same problem, showing what command(s) a flag goes with).

This typically causes me a lot of trial-and-error when plumbing in a flag to .bazelrc since I often don't know if I should use common, build, etc.

Regarding the scrolling — there is sort of already a UI for this in the sidebar table of contents, which already has highlighting.

Screenshot 2023-10-04 at 08 57 47

However, the highlighting is never updated unless clicking the sidebar (from what I can tell). So maybe it would help if just clicking anchors forced a refresh of the current section on the sidebar or something like that.

@fmeum
Copy link
Collaborator

fmeum commented Oct 4, 2023

This typically causes me a lot of trial-and-error when plumbing in a flag to .bazelrc since I often don't know if I should use common, build, etc.

@ian-h-chamberlain With 6.3.0 you can always use either common or startup (except for Starlark-defined flags, which require 6.4.0).

@iancha1992 iancha1992 added the team-Remote-Exec Issues and PRs for the Execution (Remote) team label Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 We'll consider working on this in future. (Assignee optional) team-Documentation Documentation improvements that cannot be directly linked to other team labels team-Remote-Exec Issues and PRs for the Execution (Remote) team type: documentation (cleanup)
Projects
None yet
Development

No branches or pull requests

7 participants
@jschaf @fmeum @ian-h-chamberlain @ShreeM01 @keertk @iancha1992 and others