Fix tabs not indicating activity
ClosedPublic

Authored by velan on Tue, Nov 5, 7:08 AM.

Details

Summary

This fixes the bug with tabs not indicating terminal activity.
When the tab is split and has multiple child terminals, activity in any
of the child terminals will be indicated.

BUG: 406828
FIXED-IN: 19.12.0

Test Plan

Open two tabs, run sleep 2 in the first one and switch to the other.
After the sleep ends, the tab activity is indicated just as it was before
the bug was introduced.

Diff Detail

Repository
R319 Konsole
Lint
Lint Skipped
Unit
Unit Tests Skipped
velan requested review of this revision.Tue, Nov 5, 7:08 AM
velan created this revision.
nucleo added a subscriber: nucleo.Wed, Nov 6, 8:12 PM

I see it in line above, but using qobject_cast and not checking for nullptr is waste of time, say if you're sure in type use static_cast, if not check result of qobject_cast against nullptr.

mglb retitled this revision from Fix bug 406828 to Indicate activity on tabs with splits.Thu, Nov 7, 2:07 PM
mglb edited the test plan for this revision. (Show Details)
hindenburg retitled this revision from Indicate activity on tabs with splits to Fix tabs not indicating activity.Thu, Nov 7, 2:14 PM
hindenburg edited the summary of this revision. (Show Details)
hindenburg edited the test plan for this revision. (Show Details)

Sorry I didn't see your changes before I did mine.

hindenburg accepted this revision.Thu, Nov 7, 3:05 PM

I see it in line above, but using qobject_cast and not checking for nullptr is waste of time, say if you're sure in type use static_cast, if not check result of qobject_cast against nullptr.

This line is used a number of times and can be fixed afterwards.

This revision is now accepted and ready to land.Thu, Nov 7, 3:05 PM
hindenburg closed this revision.Thu, Nov 7, 3:08 PM