8000 [Console] Aligned multiline text in vertical table by jaytaph · Pull Request #50691 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Console] Aligned multiline text in vertical table #50691

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
Jun 20, 2023

Conversation

jaytaph
Copy link
Contributor
@jaytaph jaytaph commented Jun 17, 2023
Q A
Branch? 6.4
Bug fix? no
New feature? yes
Deprecations? no
Tickets Fix #50632
License MIT
Doc PR symfony/symfony-docs#...

When using a vertical table helper in the console, any multi-line (\n) text starts on a new line underneath the header. This fixes so the extra lines are aligned with the first line in the table.

@jaytaph jaytaph requested a review from chalasr as a code owner June 17, 2023 11:07
@carsonbot carsonbot added this to the 6.4 milestone Jun 17, 2023
@jaytaph jaytaph force-pushed the ticket_50632 branch 3 times, most recently from a96e28e to a01494a Compare June 17, 2023 11:20
@jaytaph
Copy link
Contributor Author
jaytaph commented Jun 17, 2023

Seems that fabbot wants to strip away the trailing spaces in the table helper unit-tests, but in these cases they need to be present

@fabpot
Copy link
Member
fabpot commented Jun 20, 2023

Thank you @jaytaph.

@fabpot fabpot merged commit d48b929 into symfony:6.4 Jun 20, 2023
This was referenced Oct 21, 2023
nicolas-grekas added a commit that referenced this pull request Feb 20, 2024
…centLanglet)

This PR was merged into the 6.4 branch.

Discussion
----------

[Console] Fix display of vertical Table on Windows OS

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT

Similar to #54001
but the feature was introduced in 6.4 with #50691

Commits
-------

e9b0faa Fix vertical table on windows
symfony-splitter pushed a commit to symfony/console that referenced this pull request Feb 20, 2024
…centLanglet)

This PR was merged into the 6.4 branch.

Discussion
----------

[Console] Fix display of vertical Table on Windows OS

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT

Similar to symfony/symfony#54001
but the feature was introduced in 6.4 with symfony/symfony#50691

Commits
-------

e9b0faaec5 Fix vertical table on windows
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.

Multiline fields in vertical console tables
7 participants
0