Fix the QuickExtract menu
ClosedPublic

Authored by tctara on Mar 9 2016, 2:04 PM.

Details

Summary

adding directories history that users selected to the arkrc, so that QuickExtractMenu can read from and display properly.

Related :
https://.kde.org/show_bug.cgi?=357225

Diff Detail

Repository
R36 Ark
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
tctara updated this revision to Diff 2641.Mar 9 2016, 2:04 PM
tctara retitled this revision from to Fix the QuickExtract menu.
tctara updated this object.
tctara edited the test plan for this revision. (Show Details)
tctara added reviewers: elvisangelaccio, rthomsen.
tctara set the repository for this revision to R36 Ark.
tctara added a project: Ark.
tctara added a subscriber: Ark.
Restricted Application added a subscriber: kde-utils-devel. · View Herald TranscriptMar 9 2016, 2:04 PM
tctara updated this object.Mar 9 2016, 2:31 PM
rthomsen requested changes to this revision.Mar 9 2016, 6:33 PM
rthomsen edited edge metadata.

Code looks good. I tested it and seems to work properly.

kerfuffle/extractiondialog.cpp
157

Please start comments with capital letter and end them with period '.'

This revision now requires changes to proceed.Mar 9 2016, 6:33 PM
elvisangelaccio requested changes to this revision.Mar 9 2016, 6:58 PM
elvisangelaccio edited edge metadata.

Nice work, just some little changes :)

kerfuffle/extractiondialog.cpp
158

We already have an "ExtractDialog" config group name, let's re-use it. No need to create a new group with name "ExtractDialog DirHistory" ;)

159

Please call this entry "DirHistory", which is more explicative than "Item".

165

Same here, "DirHistory" instead of "Item".

part/part.cpp
453

Same here, change the name of the config group and the entry below.

tctara updated this revision to Diff 2648.Mar 9 2016, 7:00 PM
tctara updated this object.
tctara edited edge metadata.
tctara updated this revision to Diff 2649.Mar 9 2016, 7:13 PM
tctara edited edge metadata.
tctara marked an inline comment as done.
rthomsen accepted this revision.Mar 9 2016, 9:15 PM
rthomsen edited edge metadata.

Good work! :)

elvisangelaccio accepted this revision.Mar 10 2016, 5:33 PM
elvisangelaccio edited edge metadata.
This revision is now accepted and ready to land.Mar 10 2016, 5:33 PM
This revision was automatically updated to reflect the committed changes.