From feb78e15493860d8eb1226bd3315cc3549c13909 Mon Sep 17 00:00:00 2001 From: Tom Giannattasio Date: Tue, 18 Sep 2012 13:30:30 -0400 Subject: [PATCH 1/4] changed sidebar to display full post title --- lms/static/sass/_discussion.scss | 35 ++++++++++++++++++++------------ 1 file changed, 22 insertions(+), 13 deletions(-) diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss index 79c853f42d..43d5266f89 100644 --- a/lms/static/sass/_discussion.scss +++ b/lms/static/sass/_discussion.scss @@ -1052,14 +1052,17 @@ body.discussion { } a { - position: relative; display: block; - height: 36px; + position: relative; + float: left; + clear: both; + width: 100%; padding: 0 10px 0 18px; margin-bottom: 1px; margin-right: -1px; @include linear-gradient(top, rgba(255, 255, 255, .7), rgba(255, 255, 255, 0)); background-color: #fff; + @include clearfix; &:hover { @include linear-gradient(top, rgba(255, 255, 255, .7), rgba(255, 255, 255, 0)); @@ -1096,9 +1099,13 @@ body.discussion { } .title { + display: block; + float: left; + width: 70%; + margin: 8px 0 10px; font-size: 13px; font-weight: 700; - line-height: 34px; + line-height: 1.4; color: #333; } @@ -1121,7 +1128,7 @@ body.discussion { content: ''; position: absolute; top: 0; - right: 1px; + right: 0; width: 10px; height: 12px; background: url(../images/following-flag.png) no-repeat; @@ -1164,14 +1171,16 @@ body.discussion { } } - .title { - display: block; - float: left; - width: 70%; - white-space: nowrap; - text-overflow: ellipsis; - overflow: hidden; - } + // .title { + // display: block; + // float: left; + // width: 70%; + // margin: 6px 0; + // line-height: 1.4; + // white-space: nowrap; + // text-overflow: ellipsis; + // overflow: hidden; + // } .votes-count, .comments-count { @@ -1179,7 +1188,7 @@ body.discussion { float: right; width: 32px; height: 16px; - margin-top: 9px; + margin-top: 8px; border-radius: 2px; @include linear-gradient(top, #d4d4d4, #dfdfdf); font-size: 11px; From ac7562ccbf5b03f5280a2c52e8eecdd6ebb4e8ac Mon Sep 17 00:00:00 2001 From: Tom Giannattasio Date: Tue, 18 Sep 2012 13:46:25 -0400 Subject: [PATCH 2/4] added padding to vote button so titles don't hit it --- lms/static/sass/_discussion.scss | 1 + 1 file changed, 1 insertion(+) diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss index 43d5266f89..ee3383400b 100644 --- a/lms/static/sass/_discussion.scss +++ b/lms/static/sass/_discussion.scss @@ -1329,6 +1329,7 @@ body.discussion { position: relative; z-index: 100; margin-top: 5px; + margin-left: 40px; } .post-tools { From b90c37df413424c193645dcb357f4ba42db8e47a Mon Sep 17 00:00:00 2001 From: Tom Giannattasio Date: Tue, 18 Sep 2012 14:02:52 -0400 Subject: [PATCH 3/4] added styles for blockquotes and lists within responses and comments --- lms/static/sass/_discussion.scss | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss index ee3383400b..582ca9871c 100644 --- a/lms/static/sass/_discussion.scss +++ b/lms/static/sass/_discussion.scss @@ -1485,17 +1485,31 @@ body.discussion { } } + blockquote { + background: #f6f6f6; + border-radius: 3px; + padding: 5px 10px; + font-size: 14px; + } + .comments { list-style: none; margin-top: 20px; padding: 0; border-top: 1px solid #ddd; - li { + > li { background: #f6f6f6; border-bottom: 1px solid #ddd; } + blockquote { + background: #e6e6e6; + border-radius: 3px; + padding: 5px 10px; + font-size: 14px; + } + .comment-form { background: #eee; @include clearfix; @@ -1519,7 +1533,6 @@ body.discussion { .discussion-errors { margin: 0px; } - } .response-body { From 3af4ce887f1e1ec7ee8d39ac3a8e46d647dd9807 Mon Sep 17 00:00:00 2001 From: Tom Giannattasio Date: Tue, 18 Sep 2012 16:01:31 -0400 Subject: [PATCH 4/4] removed extraneous comments --- lms/static/sass/_discussion.scss | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss index 582ca9871c..4c8476f301 100644 --- a/lms/static/sass/_discussion.scss +++ b/lms/static/sass/_discussion.scss @@ -1171,17 +1171,6 @@ body.discussion { } } - // .title { - // display: block; - // float: left; - // width: 70%; - // margin: 6px 0; - // line-height: 1.4; - // white-space: nowrap; - // text-overflow: ellipsis; - // overflow: hidden; - // } - .votes-count, .comments-count { display: block;