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
Actually use the profile's blur setting instead of just echoing it.
Note that a restart is currently needed after changing the blur setting.
CCBUG: 395520
Summary:
This fixes quite a few bugs and open space for a massive cleanup.
Because the symbols where not exported we recompiled the same object
three times, one for the private library, one for the desktop
application and another for the KPart, all of those having a static
object inside of the same compilation unit - but linked in three
different objects.
This added a static data in three different objects, the object that
was supposed to be a singleton (!) - The result: KonsoleSettings::self()
in mainwindow.cpp had one pointer, in TerminalDisplay had another,
making the singleton useless.
Having just one singleton fixes a major misdesign in Konsole:
There's a call-chain starting from main.cpp that goes thru all objects
changing every setting needed, and most of the calls are just forwarding
calls to another object.
With this fixed we can remove this call-chain, and do a
connect(KonsoleSettings::self(), &KonsoleSettings::settingsChanged(),
...) inside of the object that needs to react to the changed setting
making the code much smaller and easier to figure out what's going
on.
Reviewers: hindenburg, #konsole, #kde_applications, broulik, dfaure, aacid
Reviewed By: dfaure
Subscribers: ngraham, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D13891
It was decided to implement the revertOVerlayScrollbar in breeze
so it would work for all kde apps in one go.
The Hide scrollbar when uneeded without the overlay would make the text
jump so it's also reverted.
Summary:
A common power user complaint about Plasma is the lack of a
{key Ctrl Alt T} keyboard shortcut to open Konsole. It turns out
that one has existed for quite some time in the form of an
off-by-default example for instructional purposes, shipped with
`khotkeys`.
This patch adds one such shortcut to Konsole itself, ensuring that
it will always be available when Konsole is installed.
Test Plan:
Either log into a new user account or reboot into an existing user
account; {key Ctrl Alt T} opens Konsole! It even includes the
bouncing app launch animation since Konsole opened via the
desktop file.
The shortcut itself shows up as a sub-item under the `KMenuEdit`
category in the {nav Shortcuts} KCM:
{F5950456}
Reviewers: davidedmundson, #konsole, hindenburg
Reviewed By: #konsole, hindenburg
Subscribers: #plasma, tcanabrava, konsole-devel
Tags: #konsole
Differential Revision: https://phabricator.kde.org/D13708
Kurt Hindenburg
Tomaz Canabrava
l10n daemon script
Albert Astals Cid
Martin T. H. Sandsmark
Mariusz Glebocki
Friedrich W. H. Kossebau
Nathaniel Graham