Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OS X] [Finder] Add icon to the Finder context menu #4472

Closed
gig13 opened this issue Feb 12, 2016 · 11 comments
Closed

[OS X] [Finder] Add icon to the Finder context menu #4472

gig13 opened this issue Feb 12, 2016 · 11 comments
Assignees
Labels
blue-ticket Design & UX ReadyToTest QA, please validate the fix/enhancement
Milestone

Comments

@gig13
Copy link

gig13 commented Feb 12, 2016

In Finder, the Share with ownCloud contextual menu item can get lost in the mass of options. Dropbox has an icon appear next to their menu items to make sure they stand out. I think it would be good to add that to the Mac contextual menu for ownCloud so that your eye is quickly drawn to the place you need to go when you're wanting to access that option, rather than needing to search for it.

@MTRichards @dragotin

@guruz guruz changed the title Mac client -- add Share icon to the Finder menu [OS X] [Finder] Add icon to the Finder menu Feb 14, 2016
@guruz guruz added this to the 2.2-next milestone Feb 14, 2016
@dragotin dragotin modified the milestones: backlog, 2.2-next Feb 15, 2016
@danimo
Copy link
Contributor

danimo commented Feb 15, 2016

Dupe of #296

@danimo danimo closed this as completed Feb 15, 2016
@danimo danimo removed this from the backlog milestone Feb 15, 2016
@danimo
Copy link
Contributor

danimo commented Feb 15, 2016

Moved blue ticket label there, removing here, @gig13, fyi.

@scolebrook
Copy link
Contributor

@danimo That's related to the Finder sidebar. This is related to the contextual menu for the share with ownCloud menu item.

@scolebrook scolebrook reopened this Feb 15, 2016
@guruz guruz added this to the 2.2.0-current milestone Mar 4, 2016
@guruz
Copy link
Contributor

guruz commented Mar 4, 2016

Related comment about implementation: #296 (comment)

Re-added blue ticket..

@guruz guruz changed the title [OS X] [Finder] Add icon to the Finder menu [OS X] [Finder] Add icon to the Finder context menu Mar 4, 2016
@scolebrook
Copy link
Contributor

@guruz That linked comment is related to customizing the icon of the ownCloud folder in the Finder sidebar. This issue is about the contextual menu and the Share with ownCloud menu item. It's the same situation for Windows. Dropbox has an icon next to it's menu item which makes it stand out from the clutter. If our users actually read/watched/listened to the training material we've provided for them I'd be twice as productive, but an icon in this menu would catch their eye and bring awareness of this feature. I've had numerous requests to add the ability to share a link from the desktop just like Dropbox when it's been there for ages and they just haven't seen the menu item.

The sidebar icon would be nice. But I feel this icon would not only be easier (adding an icon to a menu item is a fully supported thing) but it would have much more bang for the buck in highlighting a very useful feature. Even for those who know it's there, having the icon would make identifying the correct menu item to click on that much quicker because it'd stand out.

@guruz
Copy link
Contributor

guruz commented Mar 4, 2016

@scolebrook No worries I understood what you had said and was happy you reopened the issue. I more meant then part where @jturcotte explained that it might be tricky to get the themed icons into the Finder extension. We'll see :)

@scolebrook
Copy link
Contributor

@guruz I see. Yes. The icons for the plugin don't look like they're themed at all. Getting a themed icon available may well be harder than attaching it to the menu item. I wish I new more about cmake 😕.

@guruz guruz self-assigned this Mar 21, 2016
guruz added a commit that referenced this issue Mar 30, 2016
This works for both plain ownCloud and themed clients.
guruz added a commit that referenced this issue Mar 30, 2016
OS X: Use app icon for "Share with ..." Finder menu #4472
@guruz guruz added the ReadyToTest QA, please validate the fix/enhancement label Mar 30, 2016
@guruz
Copy link
Contributor

guruz commented Mar 30, 2016

@MTRichards @scolebrook

How hard is this to do for Windows?

I did not investigate. I guess we can just link the app icon as resource into the OCContextMenu DLL.
AFAIK Note that on Windows we use 1 DLL for all brandings so they'd fight for which icon to use. (on OS X, each branding can have a separate icon)

FYI @ogoffart @jturcotte

Then ownBrander needs an update to make this work...

We automatically use the app icon on OS X, for Windows we should do the same.

@jturcotte
Copy link
Member

We currently don't build the shell extension as part of the build on windows since we need MinGW for ownCloud but MSVC for the shell extension. The binaries are therefore pre-built and can't be customized at all by ownBrander.

Officially building binaries with MSVC would make it possible to solve it the same way as OS X, but this is not trivial to do.

@guruz
Copy link
Contributor

guruz commented Mar 30, 2016

Windows discussion --> #4627

@guruz
Copy link
Contributor

guruz commented Mar 30, 2016

Closing as @scolebrook confirmed this works for him for OS X for 2.2.x

@guruz guruz closed this as completed Mar 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blue-ticket Design & UX ReadyToTest QA, please validate the fix/enhancement
Projects
None yet
Development

No branches or pull requests

6 participants