Merge pull request #1215 from silx-kit/fix_colors

switch from BrightText to Link because on Linux BrightText is white no matter the OS theme

Author: sergey-yaroslavtsev

src/PyMca5/PyMcaGui/io/hdf5/HDF5Widget.py

@@ -320,7 +320,8 @@ def __init__(self, ffile, node, parent=None, path=None):
             #self._attrs = []
             # get the default text foreground color
             foregroundTextColor = qt.QApplication.instance().palette().color(qt.QPalette.Text)
-            NXdataColor = qt.QApplication.instance().palette().color(qt.QPalette.BrightText)
+            # qt.QPalette.Link is used to indicate action possibility (it is not actual link). BrightText and HighlightedText are not good.
+            NXdataColor = qt.QApplication.instance().palette().color(qt.QPalette.Link)
             self._color = qt.QColor(foregroundTextColor)
             if hasattr(node, 'attrs'):
                 attrs = list(node.attrs)
@@ -521,7 +522,8 @@ def data(self, index, role):
             elif role == qt.Qt.ToolTipRole:
                 item = self.getProxyFromIndex(index)
                 if hasattr(item, "color"):
-                    if item.color == qt.QApplication.instance().palette().color(qt.QPalette.BrightText) or item.color == qt.Qt.blue:
+                    # qt.QPalette.Link is used to indicate action possibility (it is not actual link). BrightText and HighlightedText are not good.
+                    if item.color == qt.QApplication.instance().palette().color(qt.QPalette.Link) or item.color == qt.Qt.blue:
                         return MyQVariant("Item has a double click NXdata associated action")
             return MyQVariant()
         except Exception:

src/PyMca5/PyMcaGui/io/hdf5/QNexusWidget.py

@@ -932,7 +932,8 @@ def hdf5Slot(self, ddict):
                             shape = None
                         self._shapeList.append(shape)
                         self.cntTable.build(self._cntList, self._aliasList, shapelist=self._shapeList)
-            elif (ddict['color'] == qt.QApplication.instance().palette().color(qt.QPalette.BrightText) or ddict['color'] == qt.Qt.blue) and ("silx" in sys.modules):
+            # qt.QPalette.Link is used to indicate action possibility (it is not actual link). BrightText and HighlightedText are not good.
+            elif (ddict['color'] == qt.QApplication.instance().palette().color(qt.QPalette.Link) or ddict['color'] == qt.Qt.blue) and ("silx" in sys.modules):
                 # there is an action to be applied
                 self.showInfoWidget(ddict["file"], ddict["name"], dset=False)
             elif ddict['type'] in ['NXentry', 'Entry']: