Skip to content

Commit

Permalink
src: ui: patch_hub: patch_hub_core: change message_box title
Browse files Browse the repository at this point in the history
The subject of the "message_box" can be either sucess or error warnings,
but in some cases the title is simply the title of the current option in
process, what makes it not homogeneous and unclear.

To manage this situation, update the "list_patches()" message_box title
to "Error".

Signed-off-by: JGBSouza <joaosouzaaa12@usp.br>
  • Loading branch information
JGBSouza committed Oct 25, 2023
1 parent ce902a4 commit f56f185
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/ui/patch_hub/patch_hub_core.sh
Original file line number Diff line number Diff line change
Expand Up @@ -160,13 +160,13 @@ function show_registered_mailing_lists()
message_box+='Select a mailing list to see the latest patchsets sent to it.'

fallback_message='kw could not find any registered mailing list.'$'\n'$'\n'
fallback_message+='Try to register mailing lists in the menu "Settings" then '
fallback_message+='Try to register mailing lists in the menu "Settings" then '
fallback_message+='"Register/Unregister Mailing Lists"'

if [[ "${#registered_mailing_lists[@]}" -eq 0 ]]; then
create_message_box 'Error' "${fallback_message}"
screen_sequence['SHOW_SCREEN']='dashboard'
return "$?"
create_message_box 'Error' "${fallback_message}"
screen_sequence['SHOW_SCREEN']='dashboard'
return "$?"
fi

create_menu_options 'Registered Mailing Lists' "$message_box" 'registered_mailing_lists' '' '' 'Return'
Expand Down Expand Up @@ -203,7 +203,7 @@ function list_patches()
local ret

if [[ -z "${_target_array_list}" ]]; then
create_message_box "${menu_title}" "${fallback_message}"
create_message_box "Error" "${fallback_message}"
screen_sequence['SHOW_SCREEN']='dashboard'
return "$?"
fi
Expand Down

0 comments on commit f56f185

Please sign in to comment.