appstreamgenerator hints that it would like a longer first paragraph. steal
the one from the documentation which seems suitably verbose and descriptive
REVIEW: 129889
This seems to be the limit in all the test files I've been trying.
Without this we get an issue with too many combining characters where
Konsole will eventually just hang.
REVIEW: 129874
For our own sanity we (falsely) assume that all surrogate characters are
double width, otherwise we have to go back and reflow the line.
This makes it work better with emojis in vim at least.
REVIEW: 129841
In previous incarnations of kuniqueapplication it used to inject a common
command option --nofork which is meant to bypass single-instance behavior.
Given that konsole can and is being invoked from scripts they may well want
to ensure that the fork they created is the actual instance of konsole.
i.e. to monitor return values and life time
Presently, since the options are divergent between konsole4 and konsole5,
scripts are either incompatible with older konsoles or with newer konsoles.
To make life easier for everyone add a compat alias --nofork, which
behaves exactly like separate.
(this unbreaks steam, which is a notable recent offender of falling into
this particular trap)
REVIEW: 129647
CHANGELOG: Added compatibility command option `--nofork` for single-instance behavior
This only addresses the vertical shift and cutoff of lower
parts of characters, but not horizontal shift (characters
moving to the left or right).
See the linked review request for details.
This commit is for the 17.04 version. If you can test master
builds, please add feedback.
REVIEW: 129281
CCBUG: 371687
Thanks to Ahmad Samir ahmadsamir3891 gmail com for patch.
Todo: any changes to the settings should auto proprogate to all tabs'
search bars.
Review: 129376
Feature: 332601
getProcessInfo() returned _foregroundProcessInfo without checking the
return value of updateForegroundProcessInfo() indicating whether
_foregroundProcessInfo was non-existent or invalid.
I think this might be the solution to a bunch of recent crash bugs,
but not closing them as I can't test.
Also skipping RB because it is a simple patch and RB is down from here.
CCBUG: 372401
CCBUG: 372620
CCBUG: 372619
CCBUG: 372593
(cherry picked from commit 40b1f0e851)
cursorRight() does not resize the _screenLines vector, leading to an
assert in QVector when trying to look up the _cuX which is bigger than
the amount of characters in the current line.
BUG: 372530
(cherry picked from commit 80b9e0775e)
getProcessInfo() returned _foregroundProcessInfo without checking the
return value of updateForegroundProcessInfo() indicating whether
_foregroundProcessInfo was non-existent or invalid.
I think this might be the solution to a bunch of recent crash bugs,
but not closing them as I can't test.
Also skipping RB because it is a simple patch and RB is down from here.
CCBUG: 372401
CCBUG: 372620
CCBUG: 372619
CCBUG: 372593
cursorRight() does not resize the _screenLines vector, leading to an
assert in QVector when trying to look up the _cuX which is bigger than
the amount of characters in the current line.
BUG: 372530
The previous set of blueish colors offered fairly bad contrast with the
charcoal grey background when using the "black" contrast value. To give
a more uniform experience the default "black" color is now charcoal grey
with it's intense variant shifted into a light grey to complement the
intense background, which is charcoal grey shifted into a more intense dark
This should offer overall better contrast, in particular in htop.
BUG: 371068
REVIEW: 129368
CHANGELOG: Breeze color scheme uses better contrasting color for "black" color value