Commit ffdf6eda authored by Randall Leeds's avatar Randall Leeds

Kill sliding-panels and summary hover shadow

The sliding panels are totally unused. They can be brought back but
right now it's unnecessary structure. The most reasonable place to
relocate the card-emphasis was in the stream-list. After all, cards
go in streams. But I found the stream-list class wasn't actually
placed on the list, it was placed on the list items. That seems
counterintuitive, so I raised it up in the structure.

Afterward, to keep the emphasis looking correct, I changed the paper
class to not have padding, and moved that padding to the thread
instead. Then I had to change some things about the thread, including
using it as a class in the editor (which is consistent, and makes
some sense). That meant that some viewer-only tag styles were bleeding
over, so I worked on those to make the tag styles a little nicer.

Finally, I had to fix the hover shadow on the threads now, and opted
to go for a background darkening instead of a shadow, since it works
across both replies and top level threads, and makes things less
bouncy, which I find to be a relief.

All in all, I'm happy. Probably will make someone unhappy.
parent 3fdce006
......@@ -566,7 +566,6 @@ blockquote {
@include smallshadow(0, 2px);
background: $white;
border: solid 1px $grayLighter;
padding: 1em;
}
......@@ -716,25 +715,23 @@ blockquote {
//Threaded discussion specific
.thread {
cursor: pointer;
padding: 1em;
position: relative;
& > ul {
padding-left: $thread-padding;
margin-left: -$thread-padding;
& > * {
border-left: 1px dotted $grayLight;
position: relative;
&:first-child {
margin-top: .5em;
}
}
}
.thread {
border-left: 1px dotted $grayLight;
height: 100%;
padding: 0;
padding-left: $thread-padding;
&:first-child {
margin-top: .5em;
}
}
.threadexp {
......@@ -761,6 +758,10 @@ blockquote {
&.collapsed {
display: block !important;
&:hover {
background-color: $grayLightest;
}
.thread {
display: none;
}
......@@ -902,22 +903,6 @@ blockquote {
}
}
//SUMMARY////////////////////////////////
//This is specific to the summary view.
.summary {
position: relative;
&:hover {
@include smallshadow(2px, 3px, .1);
bottom: 1px;
}
&:active {
@include smallshadow(2px, 1px, .1);
bottom: 0;
}
}
//STREAMER////////////////////////////////
//This is specific to the streamer page
.main-content {
......@@ -938,7 +923,13 @@ blockquote {
}
.stream-list {
margin-bottom: .75em;
& > * {
margin-bottom: .75em;
}
.card-emphasis {
@include box-shadow(6px 6px 8px -2px $grayLight);
}
}
.no-overflow {
......@@ -1118,34 +1109,23 @@ pre {outline: 1px solid #ccc; padding: 5px; margin: 5px; }
right: .1em;
top: 50%;
margin-top: -8px;
line-height: 17px;
}
.text-icon { display: none; }
input {
float: left;
position: relative;
bottom: 7px;
}
li {
cursor: pointer;
display: block;
float: left;
margin: .6em .2em;
padding: .01em 1.3em .01em .3em;
position: relative;
right: 2px;
font-size: 12px;
opacity:.8;
opacity: .8;
&:hover { opacity: 1; }
& + li { margin-left: .1em; }
&.tagit-choice { margin-top: .33em; }
&.tagit-new { padding: 0; }
& + li { margin-left: .4em; }
}
li.tagit-choice { margin-top: .33em; }
&[readonly] {
.tagit-new, .tagit-close { display: none; }
......@@ -1154,12 +1134,6 @@ pre {outline: 1px solid #ccc; padding: 5px; margin: 5px; }
}
}
// Only show one line of tags in bucket view
.summary .tags {
height: 2.0em;
overflow: hidden;
}
h3.stream {
letter-spacing: normal;
word-spacing: normal;
......
......@@ -32,42 +32,13 @@ svg { -webkit-tap-highlight-color: rgba(255, 255, 255, 0); }
}
.sliding-panels > li {
@extend .content;
@include smallshadow(-2px);
@include stretch-y;
@include transition(transform .4s);
@include translateX(100%);
height: 100%;
overflow-y: auto;
-webkit-overflow-scrolling: touch;
left: 1em;
right: 0;
&:first-child {
@include box-shadow(none);
@include translateX(0);
left: 0;
}
&.show {
@include translateX(0);
}
&.squished {
padding-left: -.5em;
}
li.card-emphasis {
box-shadow: 6px 6px 8px -2px #999999;
}
}
//SIDEBAR LAYOUT////////////////////////////////
#wrapper {
@extend .content;
height: 100%;
overflow-y: auto;
position: relative;
-webkit-overflow-scrolling: touch;
}
.topbar {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment