D28811: Add drawer on mobile and clean code

Olivier Churlaud noreply at phabricator.kde.org
Wed Apr 15 08:01:00 BST 2020


ochurlaud requested changes to this revision.
ochurlaud added a comment.
This revision now requires changes to proceed.


  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.

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200415/00399435/attachment.html>


More information about the Kde-frameworks-devel mailing list