[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-23 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #16 from David Hallas  ---
(In reply to Patrick Silva from comment #15)
> it's fixed on Arch Linux running dolphin 19.04 beta \o/

Great! Thanks for testing and commenting :D

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-23 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #15 from Patrick Silva  ---
it's fixed on Arch Linux running dolphin 19.04 beta \o/

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-22 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #14 from Patrick Silva  ---
Is this fix already available on neon dev unstable?
Bug persists on such distribution.

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-17 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #13 from Nate Graham  ---
Works great for me!

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-17 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

David Hallas  changed:

   What|Removed |Added

   Version Fixed In||19.04.0

--- Comment #12 from David Hallas  ---
Fix has been merged to Applications/19.04 and master. Please test and report
any issues :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-17 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

David Hallas  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dol
   ||phin/508dc815c75c9db8abaec2
   ||6ae664ae41bf7f89b5
 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #11 from David Hallas  ---
Git commit 508dc815c75c9db8abaec26ae664ae41bf7f89b5 by David Hallas.
Committed on 17/03/2019 at 18:11.
Pushed by hallas into branch 'Applications/19.04'.

Fix "Add Network Folder" tooltip icon does not show on Breeze, shows correctly
on Breeze Dark

Summary:
Fix "Add Network Folder" tooltip icon does not show on Breeze, shows correctly
on Breeze Dark. The fix is taken from D19596.

Test Plan:
Open Dolphin with the Breeze theme
Hover the mouse over the "Add Network Folder"
The icon is black on black

Reviewers: #dolphin, elvisangelaccio, ngraham

Reviewed By: #dolphin, elvisangelaccio, ngraham

Subscribers: broulik, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D19738

M  +17   -2src/views/tooltips/tooltipmanager.cpp

https://commits.kde.org/dolphin/508dc815c75c9db8abaec26ae664ae41bf7f89b5

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-13 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

David Hallas  changed:

   What|Removed |Added

   Assignee|dolphin-bugs-n...@kde.org   |da...@davidhallas.dk
 Status|CONFIRMED   |ASSIGNED

--- Comment #10 from David Hallas  ---
I have a fix in review here: https://phabricator.kde.org/D19738

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-12 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #9 from David Hallas  ---
(In reply to Nate Graham from comment #8)
> For reference:
> https://cgit.kde.org/kinfocenter.git/commit/
> ?id=23fa369831709f43a3100476b00ced10a1123f7f

Thanks for the quick reply :D - i'll try and take a look at it.

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-12 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #8 from Nate Graham  ---
For reference:
https://cgit.kde.org/kinfocenter.git/commit/?id=23fa369831709f43a3100476b00ced10a1123f7f

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-12 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #7 from David Hallas  ---
(In reply to Kai Uwe Broulik from comment #6)
> Doesn't it just get a QIcon/iconName? So colorizing it should be possible
> like Marco did for that other tooltip stuff just now

Hi Kai,

thanks for the reply :) Could you give some more specific pointers for this? Do
you have any working samples for this? I still haven't figured out how this is
handled for Breeze Dark, does it simply have a white version of the icon?

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-12 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=404858

--- Comment #6 from Kai Uwe Broulik  ---
Doesn't it just get a QIcon/iconName? So colorizing it should be possible like
Marco did for that other tooltip stuff just now

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-12 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=404858

Nate Graham  changed:

   What|Removed |Added

 CC||k...@privat.broulik.de

--- Comment #5 from Nate Graham  ---
Kai Uwe may be able to answer that question. :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-08 Thread David Hallas
https://bugs.kde.org/show_bug.cgi?id=404858

David Hallas  changed:

   What|Removed |Added

 CC||da...@davidhallas.dk

--- Comment #4 from David Hallas  ---
(In reply to Nate Graham from comment #3)
> The icon itself (folder-new) is fully capable of changing its color, so the
> problem here must be that Dolphin's tooltip isn't invoking that behavior
> properly.

I have been looking into this issue a bit and I can't figure out where the
"icon changing color" process should happen? I can see that Dolphin calls the
KIO::PreviewJob function, this creates a PreviewJob which ends up determining
that it should use the textthumbnail plugin to generate the thumbnail. But the
textthumbnail plugin bails out because it is a .desktop file, so the original
image is just used, which is black on white background.

So Nate, do you have some pointers to where this color changing should happen?
Could this simply be the mime type for the Add Network Location desktop entry?

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-03-02 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=404858

Patrick Silva  changed:

   What|Removed |Added

 CC||bugsefor...@gmx.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[dolphin] [Bug 404858] "Add Network Folder" tooltip icon does not show on Breeze, shows correctly on Breeze Dark

2019-02-26 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=404858

Nate Graham  changed:

   What|Removed |Added

   Assignee|visual-des...@kde.org   |dolphin-bugs-n...@kde.org
  Component|Theme - Breeze  |view-engine: tooltip
Version|master  |unspecified
Product|plasmashell |dolphin
 CC||n...@kde.org
 Ever confirmed|0   |1
   Target Milestone|1.0 |---
 Status|REPORTED|CONFIRMED

--- Comment #3 from Nate Graham  ---
The icon itself (folder-new) is fully capable of changing its color, so the
problem here must be that Dolphin's tooltip isn't invoking that behavior
properly.

-- 
You are receiving this mail because:
You are watching all bug changes.