From 4e40eaf34c578c438a2860dbe62e3a29334b4603 Mon Sep 17 00:00:00 2001 From: thankyouverycool <66646555+thankyouverycool@users.noreply.github.com> Date: Sat, 11 Sep 2021 11:30:03 -0400 Subject: [PATCH] TextEditor: Remove wordiness and a separator in View menu In many cases we can lean on the name of a menu to reduce verbosity. "View"->"Visualize/Show/Turn on/etc" is a bit redundant and clutters the menu. It's a little thing, but it makes the system feel more tightly integrated if we stick to the same word patterns across apps. --- Userland/Applications/TextEditor/MainWidget.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Userland/Applications/TextEditor/MainWidget.cpp b/Userland/Applications/TextEditor/MainWidget.cpp index a3c2353ecb..d35769861f 100644 --- a/Userland/Applications/TextEditor/MainWidget.cpp +++ b/Userland/Applications/TextEditor/MainWidget.cpp @@ -478,8 +478,6 @@ void MainWidget::initialize_menubar(GUI::Window& window) m_no_wrapping_action->set_checked(true); - view_menu.add_separator(); - m_soft_tab_width_actions.set_exclusive(true); auto& soft_tab_width_menu = view_menu.add_submenu("&Tab Width"); m_soft_tab_1_width_action = GUI::Action::create_checkable("1", [&](auto&) { @@ -514,14 +512,16 @@ void MainWidget::initialize_menubar(GUI::Window& window) view_menu.add_separator(); - m_visualize_trailing_whitespace_action = GUI::Action::create_checkable("&Visualize Trailing Whitespace", [&](auto&) { + m_visualize_trailing_whitespace_action = GUI::Action::create_checkable("T&railing Whitespace", [&](auto&) { m_editor->set_visualize_trailing_whitespace(m_visualize_trailing_whitespace_action->is_checked()); }); - m_visualize_leading_whitespace_action = GUI::Action::create_checkable("Visualize &Leading Whitespace", [&](auto&) { + m_visualize_leading_whitespace_action = GUI::Action::create_checkable("L&eading Whitespace", [&](auto&) { m_editor->set_visualize_leading_whitespace(m_visualize_leading_whitespace_action->is_checked()); }); m_visualize_trailing_whitespace_action->set_checked(true); + m_visualize_trailing_whitespace_action->set_status_tip("Visualize trailing whitespace"); + m_visualize_leading_whitespace_action->set_status_tip("Visualize leading whitespace"); view_menu.add_action(*m_visualize_trailing_whitespace_action); view_menu.add_action(*m_visualize_leading_whitespace_action);