Page MenuHome

File Browser: Exclude "AirDrop" from the "System" list (MacOS)
ClosedPublic

Authored by Yevgeny Makarov (jenkm) on Mon, Jan 13, 11:32 AM.

Details

Summary

Exclude "AirDrop" from the "System" list, if wlan not active it would show "/".
There is already code for this, it's just not working, because the slash is added before.
Just moved "add slash" down, first check if the line is "", then add a slash.

Diff Detail

Repository
rB Blender

Event Timeline

This revision is now accepted and ready to land.Mon, Jan 13, 1:02 PM

@William Reynish (billreynish) maybe the "Applications" folder should be excluded from the System list as well?

if (!strstr(line, "myDocuments.cannedSearch") && strcmp(line, "/") && strcmp(line, "/Applications/")) {

@Yevgeny Makarov (jenkm) Why should it be excluded? I mean, sure, it doesn't seem terribly useful, but IMO we should simply try and reproduce the Finder sidebar as much as we can.

It's just useless, and there's no suitable icon for it :) But let it be, I don't mind.

Yevgeny Makarov (jenkm) edited the summary of this revision. (Show Details)

Just moved "add slash" down, first check if the line is "", then add a slash.
The previous version of the patch excluded the user's "/" bookmark also.