Project

General

Profile

Actions

Todo #13268

open

Dynamically adjust tHe interface text max wdith for the login banner

Added by → luckman212 over 1 year ago. Updated 23 days ago.

Status:
Pull Request Review
Priority:
Normal
Assignee:
Category:
Console Menu
Target version:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Default

Description

small change to add some width and better align things if interface names are longer than just "WAN", "WAN2" etc.

before:

after:


Files

Actions #1

Updated by → luckman212 over 1 year ago

I wanted to auto size the columns based on the terminal width, but the shell doesn't seem to export the $COLUMNS variable that would be required to do that.

anyway- PR: https://github.com/pfsense/pfsense/pull/4598

Actions #2

Updated by Marcos M 23 days ago

  • Status changed from Ready To Test to Pull Request Review
  • Assignee set to Marcos M

The max interface description is known (31 iirc), and the real interface name is almost always relatively short, hence I think we can rely on the line-wrapping feature and not introduce a fixed width. We can calculate the max width and go with that, e.g.: Show

https://gitlab.netgate.com/pfSense/pfSense/-/merge_requests/1097

Actions #3

Updated by Marcos M 23 days ago

  • Subject changed from columns don't align nicely in console with medium-long interface names to Dynamically adjust tHe interface text max wdith for the login banner
Actions

Also available in: Atom PDF