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

[OSX ] In the columns view the overlay icons are not shown #2107

Closed
luciamaestro opened this issue Aug 26, 2014 · 7 comments
Closed

[OSX ] In the columns view the overlay icons are not shown #2107

luciamaestro opened this issue Aug 26, 2014 · 7 comments
Assignees
Labels
Enhancement ReadyToTest QA, please validate the fix/enhancement

Comments

@luciamaestro
Copy link

Expected behaviour

Overlay icons are shown in all the finder views

Actual behaviour

In the columns view overlay icons are not shown

Steps to reproduce

  1. Instal the new version
  2. Open desktop client
  3. Open the Finder select columns view

Client configuration

Version 1.7.0 nightly-2040826

Operating system: Mac 10.9.3

@guruz
Copy link
Contributor

guruz commented Oct 21, 2014

If anyone knows to do this, please comment :)
Seems like I can't find the proper hooks for the Finder.h hack.

@dekstop
Copy link

dekstop commented Nov 12, 2014

It seems there was a Nativity patch related to this shortly after you forked.
liferay/liferay-nativity@79a713e

@dragotin
Copy link
Contributor

@guruz pls quickly check if the patch mentioned above helps, if not, we should move this to 1.8

@guruz
Copy link
Contributor

guruz commented Dec 8, 2014

@dekstop Yes, I had seen that commits. Actually I was going through all the commits there and didn't see anything that would help :( Also tried around a bit, but none of our hook functions was called.

@dragotin Moving to 1.8. I begin to wonder if it is necessary, given that there is #2340

@guruz guruz modified the milestones: 1.8 - UI Enhancements, 1.7.1 Dec 8, 2014
@dragotin dragotin modified the milestones: 1.9 - Multi-account, 1.8 - UI Enhancements Jan 19, 2015
@dragotin
Copy link
Contributor

We will move to the new Mac OSX 10.10 API at some point in the future and that will fix this.

@jturcotte
Copy link
Member

Should be fixed by 0b461e2

@jturcotte jturcotte added the ReadyToTest QA, please validate the fix/enhancement label Mar 11, 2015
@luciamaestro
Copy link
Author

It is fixed in the latest version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ReadyToTest QA, please validate the fix/enhancement
Projects
None yet
Development

No branches or pull requests

5 participants