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.

Sign up for GitHub

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

Jump to bottom

Make QIcon::hasThemeIcon() work with our engine. #98

Closed
luis-pereira opened this issue Jul 30, 2016 · 0 comments
Closed

Make QIcon::hasThemeIcon() work with our engine. #98

luis-pereira opened this issue Jul 30, 2016 · 0 comments
Assignees
@luis-pereira
Labels
bug
Milestone
2.0

Comments

@luis-pereira
Copy link
Member

luis-pereira commented Jul 30, 2016 •
edited
Loading

As @paulolieuthier stated here. It's always returning true.
Makes lxqt/lxqt-panel#365 not needed.
@luis-pereira luis-pereira added this to the 2.0 milestone Jul 30, 2016
@luis-pereira luis-pereira self-assigned this Jul 30, 2016
@luis-pereira luis-pereira changed the title Make QIcon::hasIconTheme() work with our engine. Make QIcon::hasThemeIcon() work with our engine. Jul 30, 2016
luis-pereira added a commit that referenced this issue Jul 30, 2016
@luis-pereira
Fixes QIcon::hasThemeIcon() behavior with our Icon Loader Engine
bb6e6b4 
It was always returning true.
Closes #98.
Closes lxqt/lxqt#1081.
@luis-pereira luis-pereira mentioned this issue Jul 30, 2016
Merged
luis-pereira added a commit that referenced this issue Aug 5, 2016
@luis-pereira
Fixes QIcon::hasThemeIcon() behavior with our Icon Loader Engine
43930e6 
It was always returning true.
Closes #98.
Closes lxqt/lxqt#1081.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luis-pereira luis-pereira

Labels
bug
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
1 participant
@luis-pereira