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

[Console] Aligned multiline text in vertical table #50691

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
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
5 of 9 checks passed
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