Differential D7386

[AdditionalInfoDialog] Prevent hiding "text" role
ClosedPublic
Actions

Authored by broulik on Aug 18 2017, 12:27 PM.

Details

Reviewers
elvisangelaccio
Group Reviewers
Dolphin
Commits
R318:dfbaf21218c7: [AdditionalInfoDialog] Prevent hiding "text" role
Summary

You cannot do this from the menu but the option was there (but ignored) in the "View Properties" dialog.

BUG: 302622

Test Plan

"Name" is now grayed out and cannot be unchecked

Diff Detail

Repository
R318 Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Aug 18 2017, 12:27 PM
Restricted Application added a subscriber: Konqueror. · View Herald TranscriptAug 18 2017, 12:27 PM
elvisangelaccio accepted this revision.Aug 25 2017, 1:54 PM
This revision is now accepted and ready to land.Aug 25 2017, 1:54 PM
Closed by commit R318:dfbaf21218c7: [AdditionalInfoDialog] Prevent hiding "text" role (authored by broulik). · Explain WhyAug 25 2017, 2:51 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 18756

View Options

src/settings/additionalinfodialog.cpp

Loading...