Summary:
Show tabbar and hide tabbar got lost in the transition to a modern
QTabBar, this re-enables them. These options will override the
current settings if you use --hide-tabbar or --show-tabbar.
Changing these options in the settings will change these settings.
To use the current preferences just start konsole without --hide-tabbar
or --show-tabbar
Reviewers: #konsole, sandsmark, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: ngraham, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14294
Builds have been failing since this new QTabBar code was committed
since QOperatingSystemVersion::currentType() is Qt5.9+. We currently
only require Qt5.6 and I don't want to change that just for this.
ENABLE_DETACHING is only set to false for MacOSX/Darwin.
Summary:
This patch fixes the lack of profile in the new tab button,
and it makes the close button to behave the same way
as the old implementation.
Reviewers: #konsole, sandsmark, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14295
Summary:
Lambdas don't auto disconnect. Removing that code also
removed one of Nathan's nitpicks on my code.
Reviewers: #konsole, hindenburg, sandsmark
Reviewed By: #konsole, hindenburg
Subscribers: konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14298
Summary:
this is a WIP, I don't plan to merge this yet, there are many thigns to solve.
Drop the handmade TabWidget used inside of konsole in favor of Qt's QTabWidget
This drops tons of code we manually did, but it also drops a bit of functionality.
Everything besides detach should be working.
I'm now trying to finish detach.
a nice overvie:
15 files changed, 314 insertions(+), 1609 deletions(-)
Missing features:
- Detach
and lots of testing.
Reviewers: hindenburg, #konsole, ngraham
Reviewed By: ngraham
Subscribers: ngraham, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D13882
(Detaching is still broken atm)
Move out of the CMake the check for crashes in detaching
It's better to check that in the code - I was quite lost
for a while on why there was a DETACHING_ENABLED definition
as it did not made sense
Summary:
this is a WIP, I don't plan to merge this yet, there are many thigns to solve.
Drop the handmade TabWidget used inside of konsole in favor of Qt's QTabWidget
This drops tons of code we manually did, but it also drops a bit of functionality.
Missing features:
- Close Button
- New Tab on double click
- Drag'n Drop
- Detach
and lots of testing.
Subscribers: konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D13882
Summary:
There is a bug in the Qt glib event loop leading to timers never being
able to deliver signals.
Work around this by disabling the glib event loop.
References:
http://lists.qt-project.org/pipermail/interest/2015-September/018846.htmlhttps://bugreports.qt.io/browse/QTBUG-48344
Test plan:
From the referenced bug:
Stefan Westerfeld 2010-03-10 11:40:24 UTC
Running the following program within konsole:
#include <stdio.h>
int
main()
{
for (int i = 0; i < 100000000; i++)
{
fprintf (stderr, "foo %d\n", i);
}
}
leads to a freeze - not single message is printed - no reaction on
return. Only after a long time (30 seconds) something happens.
If I run the same program in an xterm, the messages are scrolling
through, as I would expect from that kind of output.
Reviewers: #konsole, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: hindenburg
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D6078
BUG: 230184
Summary:
When there was at least one search attempt, search box is still opened,
and the screen and/or history is cleared, there is a high chance that
the line number remembered by the search is invalid. The line number is
used as the lines array index, so this can lead to overflow and crash.
This is at this moment fixed with a check in `copyLineToStream()`, which
is a generic function that happens to be used by search function and
where the line number is used. There still is an `assert` which is
triggered in debug builds.
The patch moves the check directly to a search function, where the line
number is initialized before first search.
Test Plan:
You have to do the test in debug build - there is a hack in
`copyLineToStream()` which prevents crash, but `assert` before it
catches an error condition.
* Run `seq 5000`
* Open search box, search for `000`
* Clear screen/history
* Search up/down
Actual result: Crash
Expected result: Search should begin from last/first visible line
Reviewers: #konsole, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: hindenburg, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14106
Summary:
Resetting the widget palette to the app main palette fixes black
background problem in short and clean way.
Reviewers: #konsole, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: tcanabrava, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14039
Summary:
When the focus is in `IncrementalSearchBar`, unhandled keys (i.e. those
not handled by text input box or `unhandledMovementKeyPressed` signal)
are sent to a program running in the terminal, which is probably not
what user expects. Example of propagated keys: Up, Down, PgUp, PgDown,
Fn, Tab.
This patch disables propagation. Shortcuts, movement keys, and keys
handled by search bar still works.
Test Plan:
* Run `cat`
* Open search box
* Press Up/Down/PgUp/PgDown/Tab/Fn keys
Actual result: keys are sent to `cat`
Expected result: nothing should happen (implemented)
Alternative expected result: focus should move from text input to first
button
**Regression test**
Following keys/shortcuts should work:
* Shift+Up/Down/PgUp/PgDown - terminal contents scrolling
* Ctrl+Shift+T - open new tab (unless it was changed)
* Enter - search next
* Esc - close search box
Reviewers: #konsole, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14037
Summary:
Prevent search widget from moving when scrolling session.
BUG: 396119
Test Plan:
* Run `seq 1000`
* Turn on search
* Hold Shift+Up, Shift+Down
Actual result: the search box scrolls with terminal contents
Expected result: only terminal should scroll, the box should remain
unchanged
Reviewers: #konsole, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: hindenburg, ngraham, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14038
Summary:
Parent propagates the font to the child,
this was messing up the font used in the SearchBar.
Test Plan: Manually opened the search bar and verified the font.
Reviewers: hindenburg, ngraham, #konsole
Reviewed By: ngraham
Subscribers: konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D14012