Skip to content

docs: changing config-vars table to list, avoiding wrong table formatting#6594

Closed
TheWesDias wants to merge 1 commit intoaws:developfrom
TheWesDias:doc-table-format-config-vars
Closed

docs: changing config-vars table to list, avoiding wrong table formatting#6594
TheWesDias wants to merge 1 commit intoaws:developfrom
TheWesDias:doc-table-format-config-vars

Conversation

@TheWesDias
Copy link
Copy Markdown

Issue #, if available:
#6296

Description of changes:
changing config-vars tables to list, avoiding wrong table formatting

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@justindho
Copy link
Copy Markdown
Contributor

Hi @TheWesDias, thanks for the PR! I think we'd prefer to keep this in the table view if possible, as it provides a quick way for users to find what they need at a glance without having to scroll down. #4855 proposes a fix that looks like it would fix the table formatting issue. That PR is now in the "Ready to Review" stage of our new project board, and a maintainer will review the PR once we have worked through our backlog of PRs and classified them appropriately according to our new community contributions process.

For the time being, I'm going to set this PR as a draft. It is linked in the issue, so this PR will be considered as a potential solution when a maintainer reviews the issue and other PR.

@justindho justindho marked this pull request as draft May 20, 2022 19:40
@tim-finnigan
Copy link
Copy Markdown
Contributor

Closing as not planned — as mentioned above we still want to keep this in a table view. Continuing to track issue #6296

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants