Changes the tabName function to return the name of a specific tab
ClosedPublic

Authored by hallas on Aug 28 2018, 7:00 AM.

Details

Summary

Changes the tabName function to return the name of the passed in
DolphinTabPage. Previously it would return the name of the active tab,
which causes in-active tabs to get the wrong name.

Test Plan

Open Dolphin
Right click a folder and select 'Open in new tab'
Notice that the new tab has the wrong title

BUG: 397910

Diff Detail

Repository
R318 Dolphin
Branch
fixes_bug_397910
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 2428
Build 2446: arc lint + arc unit
hallas created this revision.Aug 28 2018, 7:00 AM
Restricted Application added a project: Dolphin. · View Herald TranscriptAug 28 2018, 7:00 AM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
hallas requested review of this revision.Aug 28 2018, 7:00 AM
ngraham accepted this revision as: ngraham.Aug 28 2018, 9:20 PM
ngraham added a reviewer: elvisangelaccio.

Thanks, this fixes the issue and doesn't seem to introduce any regressions in my testing.

Let's wait for Elvis this time. :)

This revision is now accepted and ready to land.Aug 28 2018, 9:20 PM

Thanks, this fixes the issue and doesn't seem to introduce any regressions in my testing.

Let's wait for Elvis this time. :)

Will do :)

elvisangelaccio accepted this revision.Sep 2 2018, 8:57 AM
elvisangelaccio added inline comments.
src/dolphintabwidget.cpp
359 ↗(On Diff #40533)

We usually use if (!tabPage)

hallas updated this revision to Diff 40847.Sep 2 2018, 9:00 AM

Changes comparison with nullptr

Changes comparison with nullptr to if (!...)

hallas marked an inline comment as done.Sep 2 2018, 9:00 AM
elvisangelaccio accepted this revision.Sep 2 2018, 9:05 AM
hallas closed this revision.Sep 2 2018, 9:08 AM