Merge lp://qastaging/~mjs7231/do-plugins/files-folders-extensions into lp://qastaging/do-plugins

Proposed by MShepanski
Status: Needs review
Proposed branch: lp://qastaging/~mjs7231/do-plugins/files-folders-extensions
Merge into: lp://qastaging/do-plugins
Diff against target: 1256 lines (+520/-479)
9 files modified
File/gtk-gui/Do.FilesAndFolders.Configuration.cs (+310/-331)
File/gtk-gui/generated.cs (+113/-113)
File/gtk-gui/gui.stetic (+17/-5)
File/src/Do/Do.FilesAndFolders/Configuration.cs (+1/-1)
File/src/Do/Do.FilesAndFolders/FileItemSource.cs (+22/-13)
File/src/Do/Do.FilesAndFolders/IndexedFolder.cs (+18/-5)
File/src/Do/Do.FilesAndFolders/IndexedFolderCollection.cs (+7/-7)
File/src/IndexPathNodeView.cs (+31/-3)
File/src/PathNodeView.cs (+1/-1)
To merge this branch: bzr merge lp://qastaging/~mjs7231/do-plugins/files-folders-extensions
Reviewer Review Type Date Requested Status
Chris Halse Rogers Pending
Review via email: mp+93903@code.qastaging.launchpad.net

Description of the change

Added option to filter folders and files by an extensions list. Includes the following updates:
  * Extensions is a simple whitelist of extensions to include (leave blank for old behavior).
  * Preferences UI updated to contain third column for extensions.
  * Preferences UI updated to include note & example about adding extension (in English).
  * Extensions list is propagated throughout the IndexedFolder classes in the code.

To post a comment you must log in.
Revision history for this message
MShepanski (mjs7231) wrote :

I can't link it like normally because they are attached to the wrong project, but the following blueprints may be solved by this update. I would argue that also including a blacklist would be a bit confusing to the UI at this point.

https://blueprints.launchpad.net/do/+spec/files-plugin-filetypes
https://blueprints.launchpad.net/do/+spec/gnome-do-file-types

Unmerged revisions

754. By MShepanski

Add option to filter files by an extensions list

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches