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

[Unimportant feature request] Changing the title of File menu-bar item? #1017

Open
tsujan opened this issue Dec 3, 2022 · 5 comments
Open

Comments

@tsujan
Copy link
Member

tsujan commented Dec 3, 2022

I was searching for a good replacement for Qps' "Command" menu-bar item and saw that QTerminal had a "File" menu-bar item. Since there is no file operation under that title, I think we might want to change it.

I know that seeing "File" as the first menu-bar item is a sort of tradition, but sometimes it's too irrelevant ;)

IMHO, "Session" would be much more relevant than "File".

@stefonarch
Copy link
Member

We need more items for a menu "File" than just "exit".
QTerminal has the preferences there, so we could at least move that, so we'd have "exit" and "preferences".

Or we find some useful commands. Is there any draft in the code left for what it was thought for initially?

@tsujan
Copy link
Member Author

tsujan commented Dec 3, 2022

Or we find some useful commands

QPs shouldn't be for running commands. It's full of strange ideas and stranger codes. It should just be cleaned up, IMO.

Let's not make @yan12125 angry by talking about Qps here ;) This report is about "File" in QTerminal.

@stefonarch
Copy link
Member

Oops, I completely misunderstood starting from your first line, thought it was about changing "command" in qps.

"Session" looks good to me.

@tsujan
Copy link
Member Author

tsujan commented Dec 3, 2022

My fault. I meant that "File" in QTerminal was as bad as "Command" in Qps.

@yan12125
Copy link
Member

yan12125 commented Dec 4, 2022

File is indeed kind of strange, and Session sounds better.

As a record, here are actions added to the File menu.

void MainWindow::setup_FileMenu_Actions()
{
menu_File->clear();
setup_Action(ADD_TAB, new QAction(QIcon::fromTheme(QStringLiteral("list-add")), tr("&New Tab"), settingOwner),
ADD_TAB_SHORTCUT, this, SLOT(addNewTab()), menu_File);
if (presetsMenu == nullptr) {
presetsMenu = new QMenu(tr("New Tab From &Preset"), this);
presetsMenu->addAction(QIcon(), tr("1 &Terminal"),
this, SLOT(addNewTab()));
presetsMenu->addAction(QIcon(), tr("2 &Horizontal Terminals"),
consoleTabulator, SLOT(preset2Horizontal()));
presetsMenu->addAction(QIcon(), tr("2 &Vertical Terminals"),
consoleTabulator, SLOT(preset2Vertical()));
presetsMenu->addAction(QIcon(), tr("4 Terminal&s"),
consoleTabulator, SLOT(preset4Terminals()));
}
menu_File->addMenu(presetsMenu);
setup_Action(CLOSE_TAB, new QAction(QIcon::fromTheme(QStringLiteral("list-remove")), tr("&Close Tab"), settingOwner),
CLOSE_TAB_SHORTCUT, consoleTabulator, SLOT(removeCurrentTab()), menu_File);
setup_Action(NEW_WINDOW, new QAction(QIcon::fromTheme(QStringLiteral("window-new")), tr("&New Window"), settingOwner),
NEW_WINDOW_SHORTCUT, this, SLOT(newTerminalWindow()), menu_File);
menu_File->addSeparator();
setup_Action(PREFERENCES, new QAction(tr("&Preferences..."), settingOwner), "", this, SLOT(actProperties_triggered()), menu_File);
menu_File->addSeparator();
setup_Action(QUIT, new QAction(QIcon::fromTheme(QStringLiteral("application-exit")), tr("&Quit"), settingOwner), "", this, SLOT(close()), menu_File);
}

@yan12125 yan12125 closed this as completed Dec 4, 2022
@yan12125 yan12125 reopened this Dec 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants