Skip to content

[Console] Update the table helper page #20941

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

Merged
merged 1 commit into from
May 8, 2025

Conversation

javiereguiluz
Copy link
Member

The page about the Table helper is like this: https://symfony.com/doc/6.4/components/console/helpers/table.html

The sidebar looks empty. I think it's missing some headers to better structure the contents into sections.

@javiereguiluz javiereguiluz added this to the 6.4 milestone May 6, 2025
@javiereguiluz javiereguiluz changed the title [Console] Updated the table helper page [Console] Update the table helper page May 6, 2025
Comment on lines -192 to +216
This code results in:
This outputs:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I saw we don't have same wording for result in this file, should we also update them ?

This outputs:
The output of this command will be:
This generates:
This will display the following table in the terminal:
This will display:
Nothing for Adding Table Titles section

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch, streamlining those things would be nice.

WDYT @symfony/docs ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! I made all this consistent and used This outputs because it's very concise and easy to understand. Thanks!

@javiereguiluz javiereguiluz merged commit 759ce7a into symfony:6.4 May 8, 2025
javiereguiluz added a commit that referenced this pull request May 8, 2025
…reguiluz)

This PR was squashed before being merged into the 7.3 branch.

Discussion
----------

[Console] Update some contents of the Tree helper

This updates the Tree helper explanation about custom styles to match the same structure as in the Table helper page (see #20941). It also adds a better example of how to create trees programmatically.

Commits
-------

2727dde [Console] Update some contents of the Tree helper
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.

4 participants