abstractdevelop added a comment.

  Sorry everybody for not seeing all the new comments sooner! :D
  
  Ok, it seems like the some corners being rounded is a common complaint, so we 
should address it. I agree that it looks a little out of place in System 
Settings. The only trouble is doing this in a way which respects the line at 
the top.
  
  As for the inactive tab background, the problem is that I can't seem to find 
a QPalette color which is not too light/dark. The current one 
(QPalette::Disabled, QPalette::Window) is the best I could find, but we can go 
back to using the rather extreme dark one as is used in the current tabs, which 
IMO doesn't look good with this style. We could also make the current tab 
background *lighter* instead, as in manueljlins mockups where it is almost 
white. But that appears to be an issue left to colorshemes and not to the code. 
;)

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D29264

To: abstractdevelop, #vdg, #breeze
Cc: manueljlin, filipf, ngraham, cblack, plasma-devel, dmenig, Orage, 
LeGast00n, The-Feren-OS-Dev, konkinartem, ian, jguidon, Ghost6, jraleigh, 
zachus, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, 
trickyricky26, ragreen, crozbo, ndavis, ZrenBot, firef, skadinna, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, aaronhoneycutt, abetts, sebas, apol, 
ahiemstra, mbohlender, mart

Reply via email to