davidhurka added a comment.

  The UI looks good and I understand the code. :)
  
  For dark color themes: maybe the color of the adjacent text would be good as 
foreground color for the icons. Don’t know, but maybe 
QGuiApplication::pallete().color(QPalette::WindowText)?

REPOSITORY
  R223 Okular

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

To: tobiasdeiminger, #okular
Cc: davidhurka, knambiar, ngraham, okular-devel, joaonetto, tfella, darcyshen, 
aacid

Reply via email to