I like the aim I had a few years ago: everything should be fast, and as small as possi

[help wanted] Remove useless things. Make this package intuitive. about filemanager HOT 3 OPEN

math2001 commented on May 31, 2024

[help wanted] Remove useless things. Make this package intuitive.

from filemanager.

Comments (3)

math2001 commented on May 31, 2024

Reorganize the settings files according to:

The discoverability of an option should be proportional to the distance between the perfect solution and the current solution

FileManager/FileManager.sublime-settings

Lines 2 to 3 in 4ecc88d

    
           // auto complete with files 
        
           "complete_with_files_too": true,

What does that even do?

FileManager/FileManager.sublime-settings

Lines 5 to 9 in 4ecc88d

    
           // only relevant if complete_with_files_too is true 
        
           // choose folder over file if folder and file are available for 
        
           // completion. Null means that it will be selected by alphabetic order 
        
           // valid values: "files", "folders", "alphabetic" 
        
           "pick_first": "folders",

name too short. I have to read the comment to know what it does (i should only have to read the comment to learn how to use it, if I can't "feel" that. Ideally, I shouldn't need comment, but come on, I'm not smart enough for that).

FileManager/FileManager.sublime-settings

Lines 11 to 12 in 4ecc88d

    
           // define if the auto completion case sensitive 
        
           "case_sensitive": false,

name too short. Same as above

FileManager/FileManager.sublime-settings

Lines 14 to 15 in 4ecc88d

    
           // recommended to be a char that cannot be in a file name 
        
           "index_folder_separator": ">",

I don't know what this does, even reading at the comment.

FileManager/FileManager.sublime-settings

Lines 17 to 19 in 4ecc88d

    
           // which folder to pick for reference by default 
        
           // (create folder from it) 
        
           "default_index": 0,

why do I have to think about that?? Of course I'm going to pick 0. name to short. Same as above.

FileManager/FileManager.sublime-settings

Lines 21 to 25 in 4ecc88d

    
           // valid value 
        
           // false: disable the log 
        
           // "user": display a user friendly path. eg: ~/Desktop/   (working on window) 
        
           // "computer": display a computer friendly path: C:\User\<user>\Desktop\ 
        
           "log_in_status_bar": "computer",

this shouldn't be an option. Remove it. Choose for the user. Always log the computer friendly path.

FileManager/FileManager.sublime-settings

Lines 27 to 58 in 4ecc88d

    
           // terminals 
        
           // if there is only one, it will directly open it 
        
           // otherwise, it will open a quick panel with all 
        
           // the name listed 
        
           // example for cmder: 
        
           // { "name": "Cmder", "cmd": ["C:/cmder/cmder.exe", "/SINGLE", "$cwd"] } 
        
           // $cwd will be replaced by the current working directory 
        
           "terminals": [ 
        
               { 
        
                   "name": "CMD", 
        
                   "cmd": ["cmd"], 
        
                   "platform": "windows" 
        
               }, 
        
               { 
        
                   "name": "Terminal", 
        
                   "cmd": ["open", "-a", "Terminal", "$cwd"], 
        
                   "platform": "osx" 
        
               }, 
        
               { 
        
                   "name": "iTerm", 
        
                   "cmd": ["open", "-a", "iTerm", "$cwd"], 
        
                   "platform": "osx" 
        
               }, 
        
               { 
        
                   "name": "GNOME Terminal", 
        
                   "cmd": ["gnome-terminal"], 
        
                   "platform": "linux" 
        
               } 
        
           ],

good, it's intuitive to use, except for platform which should be called platforms and take a list. The comments should be clearer and more "compact", it doesn't feel like they belong to the terminals setting.

FileManager/FileManager.sublime-settings

Lines 60 to 62 in 4ecc88d

    
           // If set to true, all the command that are disabled (in grey) 
        
           // will be hidden 
        
           "menu_without_distraction": true,

I like that. Just reading the name I know I want that on. So that probably means it shouldn't be an option in the first place. But it's kind a cute. Should be removed for sure. 🙁

FileManager/FileManager.sublime-settings

Lines 64 to 65 in 4ecc88d

    
           // no need to comment anything on that one I guess :) 
        
           "auto_close_empty_groups": false,

Remove "funny" comments please. It's not even that obvious.

FileManager/FileManager.sublime-settings

Lines 67 to 70 in 4ecc88d

    
           // auto refresh the side bar when you run any action that might affect it 
        
           // by default, sublime text would do it by itself, but if this is 
        
           // set to true, then it will be explicitly refreshed 
        
           "explicitly_refresh_sidebar": false,

Is it expensive to always have that on? And by default it's off, and I've never had any problem... I wonder if some people have it on... So maybe it should be removed.

FileManager/FileManager.sublime-settings

Lines 72 to 74 in 4ecc88d

    
           // if true, each time you create/rename/duplicate etc a file, it will be revealed 
        
           // in the sidebar 
        
           "reveal_in_sidebar": false,

ok that's probably the best one right now. The name explain what it does. The comment is nearly perfect, it could be better if it used a different wording than the setting name (ie. reveal)

FileManager/FileManager.sublime-settings

Lines 76 to 77 in 4ecc88d

    
           // Save after creating a file (because a snippet can be inserted) 
        
           "save_after_creating": false,

gooood. The comment should give case-scenario example, it's not the clearest (even for me).

FileManager/FileManager.sublime-settings

Lines 79 to 83 in 4ecc88d

    
           "aliases": { 
        
               "st": "$packages", 
        
               "des": "~/Desktop", 
        
               "here": "$file_path" 
        
           },

Perfect? Don't even need a comment saying you can use aliases within aliases because it does it in the default value, so the user can understand it off the bat. Maybe have a link to the list of default aliases in there though. And explicitly say that these aliases will be added to the default aliases.

FileManager/FileManager.sublime-settings

Lines 85 to 86 in 4ecc88d

    
           // See https://math2001.github.io/FileManager/aliases/#watch-out-for-infinite-loops 
        
           "open_help_on_alias_infinite_loop": true,

pretty sweet, except the link's broken... The comment should indicate that it'll just open the browser for help.

FileManager/FileManager.sublime-settings

Lines 88 to 106 in 4ecc88d

    
           // Once again, to improve your speed, if there is commands 
        
           // you never use, you can super easily hide them. 
        
           // You can hide/show every command, and it will have no impact 
        
           // on the other ones. 
        
           "show_create_command": true, 
        
           "show_copy_command": true, 
        
           "show_delete_command": true, 
        
           "show_duplicate_command": true, 
        
           "show_edit_to_the_left_command": true, 
        
           "show_edit_to_the_right_command": true, 
        
           "show_find_in_files_command": true, 
        
           "show_move_command": true, 
        
           "show_open_in_explorer_command": true, 
        
           "show_open_in_browser_command": true, 
        
           "show_open_terminal_command": true, 
        
           "show_rename_command": true, 
        
           "show_create_from_selection_command": true,

I doubt anyone would decide to suddenly "hide" an option. If they do, then maybe that's a sign that it should be removed completely from file manager (a good indicator that I should remove it is me saying "to improve your speed". Don't aim to improve your speed by customizing your plugins. You're wasting your time. Code stuff). But it's cute. Remove it? 😢

FileManager/FileManager.sublime-settings

Lines 108 to 109 in 4ecc88d

    
           // Set to false to disable the default alt+n key binding 
        
           "create_keybinding_enabled": true

Bad name (no one thinks of "New..." as "create"), but good comment.

from filemanager.

mwchase commented on May 31, 2024

FileManager/FileManager.sublime-settings

Lines 67 to 70 in 4ecc88d

// auto refresh the side bar when you run any action that might affect it

// by default, sublime text would do it by itself, but if this is

// set to true, then it will be explicitly refreshed

"explicitly_refresh_sidebar": false,

Is it expensive to always have that on? And by default it's off, and I've never had any problem... I wonder if some people have it on... So maybe it should be removed.

I don't know if you mean "remove and act like it's defaulted to true" or "remove and act like it's defaulted to false", but for what it's worth, having it act like it's off (#54) makes the New file/folder stuff really annoying to use, and I haven't yet noticed issues from editing my local copy to force it on.

from filemanager.

TerminalFi commented on May 31, 2024

FileManager/FileManager.sublime-settings

Lines 67 to 70 in 4ecc88d

// auto refresh the side bar when you run any action that might affect it

// by default, sublime text would do it by itself, but if this is

// set to true, then it will be explicitly refreshed

"explicitly_refresh_sidebar": false,

Is it expensive to always have that on? And by default it's off, and I've never had any problem... I wonder if some people have it on... So maybe it should be removed.

I don't know if you mean "remove and act like it's defaulted to true" or "remove and act like it's defaulted to false", but for what it's worth, having it act like it's off (#54) makes the New file/folder stuff really annoying to use, and I haven't yet noticed issues from editing my local copy to force it on.

What version of ST are you using? And you are saying that it is annoying to have it disabled?

from filemanager.

[help wanted] Remove useless things. Make this package intuitive. about filemanager HOT 3 OPEN

Comments (3)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	// only relevant if complete_with_files_too is true
	// choose folder over file if folder and file are available for
	// completion. Null means that it will be selected by alphabetic order
	// valid values: "files", "folders", "alphabetic"
	"pick_first": "folders",

	// define if the auto completion case sensitive
	"case_sensitive": false,

	// recommended to be a char that cannot be in a file name
	"index_folder_separator": ">",

	// which folder to pick for reference by default
	// (create folder from it)
	"default_index": 0,

	// valid value
	// false: disable the log
	// "user": display a user friendly path. eg: ~/Desktop/ (working on window)
	// "computer": display a computer friendly path: C:\User\<user>\Desktop\
	"log_in_status_bar": "computer",

	// terminals
	// if there is only one, it will directly open it
	// otherwise, it will open a quick panel with all
	// the name listed
	// example for cmder:

	// { "name": "Cmder", "cmd": ["C:/cmder/cmder.exe", "/SINGLE", "$cwd"] }

	// $cwd will be replaced by the current working directory

	"terminals": [
	{
	"name": "CMD",
	"cmd": ["cmd"],
	"platform": "windows"
	},
	{
	"name": "Terminal",
	"cmd": ["open", "-a", "Terminal", "$cwd"],
	"platform": "osx"
	},
	{
	"name": "iTerm",
	"cmd": ["open", "-a", "iTerm", "$cwd"],
	"platform": "osx"
	},
	{
	"name": "GNOME Terminal",
	"cmd": ["gnome-terminal"],
	"platform": "linux"
	}
	],

	// If set to true, all the command that are disabled (in grey)
	// will be hidden
	"menu_without_distraction": true,

	// no need to comment anything on that one I guess :)
	"auto_close_empty_groups": false,

	// auto refresh the side bar when you run any action that might affect it
	// by default, sublime text would do it by itself, but if this is
	// set to true, then it will be explicitly refreshed
	"explicitly_refresh_sidebar": false,

	// if true, each time you create/rename/duplicate etc a file, it will be revealed
	// in the sidebar
	"reveal_in_sidebar": false,

	// Save after creating a file (because a snippet can be inserted)
	"save_after_creating": false,

	"aliases": {
	"st": "$packages",
	"des": "~/Desktop",
	"here": "$file_path"
	},

	// See https://math2001.github.io/FileManager/aliases/#watch-out-for-infinite-loops
	"open_help_on_alias_infinite_loop": true,

	// Once again, to improve your speed, if there is commands
	// you never use, you can super easily hide them.

	// You can hide/show every command, and it will have no impact
	// on the other ones.

	"show_create_command": true,
	"show_copy_command": true,
	"show_delete_command": true,
	"show_duplicate_command": true,
	"show_edit_to_the_left_command": true,
	"show_edit_to_the_right_command": true,
	"show_find_in_files_command": true,
	"show_move_command": true,
	"show_open_in_explorer_command": true,
	"show_open_in_browser_command": true,
	"show_open_terminal_command": true,
	"show_rename_command": true,
	"show_create_from_selection_command": true,

	// Set to false to disable the default alt+n key binding
	"create_keybinding_enabled": true