simgunz added a comment.

  So in createAppearanceWidget() we create a QFormLayout (instead that a 
QGridLayout) and then this is passed down to all the createStyleWidget methods, 
is that ok?
  
  I think it needs to be passed down, instead of having independents 
QFormLayouts for each annotation in order to have the Color and Opacity 
elements correctly aligned with the elements below ( which I think won't happen 
if we have two separate QFormLayouts on the top and on the bottom).

REPOSITORY
  R223 Okular

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

To: simgunz, #okular, #vdg, ngraham
Cc: aacid, okular-devel, knambiar, ngraham, joaonetto, tfella, darcyshen

Reply via email to