New tabbar implementation (accidentally?) added small margin between
terminal and tabbar's bottom separating line. This patch removes the
margin.
Comparison:
Depends on D17605
tcanabrava | |
ndavis | |
hindenburg |
Konsole | |
VDG |
New tabbar implementation (accidentally?) added small margin between
terminal and tabbar's bottom separating line. This patch removes the
margin.
Comparison:
Depends on D17605
Tested with default style and custom stylesheet
No Linters Available |
No Unit Test Coverage |
Buildable 6055 | |
Build 6073: arc lint + arc unit |
What would be the result if the cás was replaced by serDocumentMode(true); ?
Em sáb, 15 de dez de 2018 às 18:23, Mariusz Glebocki <
noreply@phabricator.kde.org> escreveu:
mglb updated this revision to Diff 47629.
mglb added a comment. View Revision https://phabricator.kde.org/D17607Remove redundant argument from setCss()
*CHANGES SINCE LAST UPDATE*
https://phabricator.kde.org/D17607?vs=47628&id=47629*BRANCH*
arc/Restore-previous-tabbar-look (branched from master)*REVISION DETAIL*
https://phabricator.kde.org/D17607*AFFECTED FILES*
src/ViewContainer.cpp
src/ViewContainer.h*To: *mglb, Konsole, VDG
*Cc: *konsole-devel, thsurrel, ngraham, maximilianocuria, hindenburg
There is setDocumentMode(true); in constructor already. I didn't find any non-css way to remove it.
I kinda like the line, but in a way I can't really explain, and not enough to object to this patch. :)
There's still a line, there's just no margin. It looks especially good with Breeze Dark:
I have (also) always trouble to see which tab is the active, at least with Dark theme. Would be nice you improve this even more.
@loh.tar I agree this looks bad with so small (relatively) lines. But this is a thing for separate request for VDG. In the meantime, search for konsole stylesheets.