nibags added a comment.

  I also like `hasXXOverride()`.
  
  In the case of **bold**, **italic**, **underline** and **strikeout**, 
functions such as `hasXXOverride()` are needed, since they are boolean values 
that can be turned off. But for the case **color**, **selectedColor**, 
**backgroundColor** and **selectedBackgroundColor**, do you think it's also 
necessary?
  
  Because I think it's better not to add those functions; with the already 
existing functions `hasTextColor(Theme)` and `hasBackgroundColor(Theme)` is 
enough.

REPOSITORY
  R216 Syntax Highlighting

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

To: nibags, #framework_syntax_highlighting, dhaumann, cullmann, vkrause
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, 
ngraham, bruns, demsking, cullmann, sars, dhaumann

Reply via email to