summaryrefslogtreecommitdiffstats
path: root/usr.sbin/bsdconfig/console/ttys
diff options
context:
space:
mode:
authordteske <dteske@FreeBSD.org>2013-06-02 20:02:50 +0000
committerdteske <dteske@FreeBSD.org>2013-06-02 20:02:50 +0000
commite9cea3453922d1975e0825166772ffe465661dab (patch)
tree2c9bf92c5092f58cc207f2fd3418feab48459fea /usr.sbin/bsdconfig/console/ttys
parentdb074b4e6ec9f60ba7d39270d7bd41bd4d5f6ba8 (diff)
downloadFreeBSD-src-e9cea3453922d1975e0825166772ffe465661dab.zip
FreeBSD-src-e9cea3453922d1975e0825166772ffe465661dab.tar.gz
Standardize the way functions build their arguments leading up to a dialog
invocation. Specifically, "top-load" your arguments and in the order in- which they will be displayed. For example, many [if not all] widgets display information in the following order, top-to-bottom (visually): + backtitle (displayed behind the widget at top-left) + title (at the top of the `window') + prompt text (just below the title and above whatever widget you choose) + Depending on widget, _one_ of the following: - menu list - radio list - check list - text input box with initial text - [Xdialog(1)] 2x or 3x text input boxes - [dialog(1)] a multi-part form - progress bar - etc. (many more widget choices) + buttons (right below the selected widget) + [dialog(1)] the hline (displayed at bottom of `window') NOTE: Xdialog(1) accepts and silently ignores --hline When building local arguments for your dialog invocation, if the value can't be cleanly loaded into a local, add "# Calculated below" to the end of the local declaration while retaining the block order of argument declarations. Move other local declarations that are not associated with this top-loading the dialog arguments to right-above where they are first-used. Also, standardize on the names of the arguments. For example, always use $prompt (instead of sometimes $msg and sometimes $prompt); use $menu_list or $shell_list or $radio_list for those respective widgets; ad nauseum. While we're doing this, flush-out full arguments for many invocations that were passing NULL strings (making it unapparent if you were staring at this one invocation what argument that NULL string was supposed to represent). Last, while we're in startup/rcconf let's remove the unnecessary use of a GLOBAL (RCCONF_MENU_LIST) for the menu_list.
Diffstat (limited to 'usr.sbin/bsdconfig/console/ttys')
-rwxr-xr-xusr.sbin/bsdconfig/console/ttys2
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.sbin/bsdconfig/console/ttys b/usr.sbin/bsdconfig/console/ttys
index 96126ad..f488efd 100755
--- a/usr.sbin/bsdconfig/console/ttys
+++ b/usr.sbin/bsdconfig/console/ttys
@@ -72,8 +72,8 @@ TTY_MENU_LIST="
#
dialog_menu_main()
{
- local hline="$hline_choose_a_terminal_type"
local prompt="$msg_ttys_menu_text"
+ local hline="$hline_choose_a_terminal_type"
local height width rows
eval f_dialog_menu_size height width rows \
OpenPOWER on IntegriCloud