Fixed new tab from preset and context menu's collapse action

lxqt / qterminal

A lightweight Qt-based terminal emulator

https://lxqt.github.io

GNU General Public License v2.0

611 stars 153 forks source link

Closed tsujan closed 3 weeks ago

tsujan commented 3 weeks ago

This patch fixes two bugs when the terminal is started with a preset of N > 1 terminals:

Previously, the action "New tab from preset" didn't work correctly when the number of preset was less than N.
Also, if the terminal context menu was the first menu that was opened, the action "Collapse Subterminal" would be disabled.

I also silenced a compilation warning about &QCheckBox::stateChanged.

tsujan commented 3 weeks ago

@stefonarch I think it's better to remove these bugs before the next release; hence bothering you with testing. I've tested it too.