gmarull / qtmodern

PyQt/PySide Widgets Modern User Interface
MIT License
751 stars 113 forks source link

Added stylesheet entry for QTableWidget #46

Closed razaqq closed 4 years ago

razaqq commented 4 years ago

grafik

Jerakin commented 4 years ago

I am not sure about using QPalette.LinkVisited for this. It should be QPalette.Base, but changing that would rightfully influence the whole gui and make it extremly flat.

How come you would like this change? Is it because when using alternate rows it looks bad? If that is the case I would think it would be better to change the QPalette.AlternateBase to work better with the rest of the colors.

razaqq commented 4 years ago

I am not sure about using QPalette.LinkVisited for this. It should be QPalette.Base, but changing that would rightfully influence the whole gui and make it extremly flat.

How come you would like this change? Is it because when using alternate rows it looks bad? If that is the case I would think it would be better to change the QPalette.AlternateBase to work better with the rest of the colors.

I find the default row colors way too dark. Making them slightly more bright makes it much easier to read. I used QPalette.LinkVisited, because it was the only free palette item not being used and i did not want to change any of the other ones.

razaqq commented 4 years ago

on light mode: grafik

Jerakin commented 4 years ago

Sweet! The dark themes is looking good but I think that the difference between the rows in the light theme is a bit too much. What do you think of 222, 222, 222 as a color?

razaqq commented 4 years ago

sounds good, changed

Jerakin commented 4 years ago

Awesome! Thanks for the fixes