D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-27 Thread Nathaniel Graham
ngraham added a dependent revision: D29238: Use Standard "Show/Hide Hidden files" shortcuts in directory chooser dialog. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D28802 To: ngraham, dfaure, #frameworks Cc: elvisangelaccio, kde-frameworks-devel, LeGast00n, cblack,

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-20 Thread Nathaniel Graham
ngraham added a comment. I pondered that myself but every app needs to implement its own behavior, and we're not even consistent about making this a toggle action vs two separate actions. That alone is probably a good reason to make a standard action, though. REPOSITORY R237 KConfig

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-20 Thread Elvis Angelaccio
elvisangelaccio added a comment. Does it make sense to also create a standard action in kconfigwidgets? REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D28802 To: ngraham, dfaure, #frameworks Cc: elvisangelaccio, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngr

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-20 Thread Elvis Angelaccio
elvisangelaccio added a comment. > and F8 seems kind of random. I was asking myself why F8 was chosen and yes, it looks like it was totally random: https://github.com/KDE/kdelibs/commit/8e26f7410a9280b504d97cdfbe5d3568cfa7b9d4 REPOSITORY R237 KConfig REVISION DETAIL https://phabric

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-13 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes. Closed by commit R237:65cc12ab3ec8: Add standard shortcut for "Show/Hide Hidden Files" (authored by ngraham). REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28802?vs=80039&id=80049 REVI

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-13 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R237 KConfig BRANCH add-show-hide-hidden-files-shortcut (branched from master) REVISION DETAIL https://phabricator.kde.org/D28802 To: ngraham, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-13 Thread Nathaniel Graham
ngraham edited the test plan for this revision. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D28802 To: ngraham, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-13 Thread Nathaniel Graham
ngraham added a dependent revision: D28803: [KDirOperator] Use new "Show/Hide Hidden Files" standard shortcut. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D28802 To: ngraham, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-13 Thread Nathaniel Graham
ngraham added a dependent revision: D28804: Use new "Show/Hide Hidden Files" standard shortcut. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D28802 To: ngraham, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28802: Add standard shortcut for "Show/Hide Hidden Files"

2020-04-13 Thread Nathaniel Graham
ngraham created this revision. ngraham added reviewers: dfaure, Frameworks. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ngraham requested review of this revision. REVISION SUMMARY Add a standard shortcut for this action so it can be synchronized between