ognarb planned changes to this revision.
ognarb added a comment.

  In D28811#648700 <https://phabricator.kde.org/D28811#648700>, @ochurlaud 
wrote:
  
  > I really dislike how you close brackets in css : it's more dense and harder 
to read.
  >
  > You indent sometimes with 1,2 or 3 spaces: it should be consistent.
  >
  > Is it something common to have css declaration within others? It's the 
first time I see that, so I'm confused.
  
  
  The css code is generated from the new scss file. I didn't remove the 
generated CSS code because I didn't want to add sass as a dependency of 
kapidox. I will add this information to the doc.
  
  The command to generate the css code is:
  
    sass src/kapidox/data/htmlresource/css/kapidox.scss 
src/kapidox/data/htmlresource/css/kapidox.css
  
  Maybe this file can be moved later to `websites/aether-sass` so that the 
layout with the sidebar can be shared with `websites/aether-sphinx` and 
`websites/aether-mediawiki` and hosted in the KDE cdn.

REPOSITORY
  R264 KApiDox

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

To: ognarb, cblack, ochurlaud
Cc: kde-frameworks-devel, kde-doc-english, LeGast00n, cblack, gennad, 
fbampaloukas, michaelh, ngraham, bruns, skadinna

Reply via email to