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

Add `-RepeatHeader` to `Format-Table` to enable repeating header for each screen full #8481

Merged
merged 7 commits into from Dec 20, 2018

Conversation

@SteveL-MSFT
Copy link
Member

commented Dec 18, 2018

PR Summary

When using a screen reader or just getting the output of a table with lots of rows, the header is no longer on the screen and the columns may no longer make sense without the context. This PR adds a -RepeatHeader switch to Format-Table to enable re-outputting the header after every screen full (minus 1 row). Expectation is that the user is piping the output to a pager (e.g. less) which uses the bottom row for pager information. I followed the AutoSize parameter as the way to get the parameter from the cmdlet into the deep formatting object where it's needed.

gifv

Fix #8455

PR Checklist

@SteveL-MSFT SteveL-MSFT force-pushed the SteveL-MSFT:format-table-header branch from 4b4f8b3 to efe8124 Dec 18, 2018
@SteveL-MSFT

This comment has been minimized.

Copy link
Member Author

commented Dec 18, 2018

The remaining codefactor issues are not related to this PR or I'm adopting the current conventions in the source code for consistency but breaks CodeFactor rules.

@SteveL-MSFT SteveL-MSFT force-pushed the SteveL-MSFT:format-table-header branch 2 times, most recently from 1073566 to 38c5f8e Dec 18, 2018
@SteveL-MSFT SteveL-MSFT force-pushed the SteveL-MSFT:format-table-header branch from 38c5f8e to 3fc6463 Dec 18, 2018
SteveL-MSFT and others added 3 commits Dec 19, 2018
Propose a different way to organize the code
@daxian-dbw daxian-dbw merged commit 110f78b into PowerShell:master Dec 20, 2018
7 checks passed
7 checks passed
CodeFactor 21 issues fixed. 19 issues found.
Details
PowerShell-CI-linux #PR-8481-20181219.02 succeeded
Details
PowerShell-CI-macos #PR-8481-20181219.02 succeeded
Details
PowerShell-CI-spelling #PR-8481-20181219.02 succeeded
Details
PowerShell-CI-windows #PR-8481-20181219.02 succeeded
Details
WIP Ready for review
Details
license/cla All CLA requirements met.
Details
@iSazonov iSazonov added the CL-Engine label Jan 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.