-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(ecep-113): style news & related patterns #16
Changes from all commits
b86b86d
253e606
c0bd5ce
5263a3a
19d54fb
d896aa3
cd56a6d
22b324d
0d1c4b3
bf07165
81eb30b
8ab2a5b
21c3a2d
ef93be3
4dbc85f
370220a
4e04181
10bbe57
6052e72
33a6d3b
d685359
6c555f9
efdc213
172c32a
156aee2
ebbfa02
0045a18
0830829
c5d5c8f
a1d6fe3
00aa175
02cdc67
7d11703
27d58cf
c896cce
8c0c594
65c643c
b3bcaeb
95f6618
d3566fb
bb328a7
0b39df8
9b8ec52
c6886b6
54ebdac
1b4416a
a00240f
a92c247
7403cbc
51eab20
d0a24e2
b71062c
ca82444
8f79d0d
99fb122
cd14d69
213b29a
f2d6a70
9136bc9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
/* | ||
Figure (Bootstrap) | ||
|
||
Add to Bootstrap styles. See: | ||
|
||
- [Bootstrap Figures](https://getbootstrap.com/docs/4.0/content/figures/) | ||
|
||
Styleguide Components.Bootstrap.Figure | ||
*/ | ||
@import url("_imports/elements/figure.css"); | ||
|
||
.figure { | ||
@extend figure; | ||
} | ||
.figure-caption { | ||
@extend figcaption; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
/* | ||
Pagination (Bootstrap) | ||
|
||
Style Bootstrap pagination. See: | ||
|
||
- [Bootstrap Pagination](https://getbootstrap.com/docs/4.0/components/pagination/) | ||
|
||
Styleguide Components.Bootstrap.Pagination | ||
*/ | ||
@import url("@tacc/core-styles/source/_imports/components/c-page.css"); | ||
|
||
|
||
|
||
/* CAVEAT: These rules only style a minimal instance of Bootstrap pagination */ | ||
/* SEE: https://github.com/nephila/djangocms-blog/blob/1.1.1/djangocms_blog/templates/djangocms_blog/post_list.html */ | ||
.pagination { | ||
@extend .c-page-list; | ||
|
||
margin-block: var(--global-space--x-large); | ||
|
||
width: fit-content; | ||
margin-inline: auto; | ||
} | ||
.pagination a, | ||
.pagination span { | ||
@extend .c-page-end; | ||
|
||
padding-block: 0; | ||
} |
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
/* | ||
Figure & Figcaption (Text Content) | ||
|
||
Element that represents self-contained content, potentially with an optional caption. | ||
|
||
Reference: | ||
|
||
- [HTML: Figure | MDN](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/figure) | ||
- [HTML: Figcaption | MDN](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/figcaption) | ||
|
||
Styleguide Elements.TextContent.Figure | ||
*/ | ||
|
||
figure > img { | ||
margin-bottom: 20px; | ||
} | ||
figcaption { | ||
color: var(--global-color-primary--dark); | ||
font-size: var(--global-font-size--medium); | ||
border-top: var(--global-border-width--normal) solid var(--global-color-primary--normal); | ||
padding-top: 12px; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -110,6 +110,10 @@ Reference: | |
Styleguide Elements.TextContent | ||
*/ | ||
|
||
p { | ||
margin-bottom: 2rem; /* overwrite Bootstrap */ | ||
} | ||
|
||
Comment on lines
+113
to
+116
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note. This style—and this whole file—could be considered CMS-specific, in which case it should move to Core-CMS via TACC/Core-CMS#482, but I just haven't confirmed whether either (this code or this whole file) is CMS-specific yet, because the Docs could also use such basic html element styles. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Seems reasonable to leave it here until you are sure. =) |
||
/* Lists */ | ||
|
||
ol, ul, | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,9 +7,17 @@ Styleguide Objects.OffsetContent | |
*/ | ||
@import url("_imports/tools/media-queries.css"); | ||
|
||
[class*="o-offset-content--"] { | ||
--offset-distance: 8.5vw; /* NOTE: Value is from Texascale.org 2020 */ | ||
--buffer: 30px; /* double Bootstrap `.col` padding */ | ||
|
||
|
||
@custom-selector :--o-offset-content | ||
.o-offset-content--left, | ||
.o-offset-content--right; | ||
|
||
|
||
|
||
:--o-offset-content { | ||
--offset-distance: min(30vw, 410px); | ||
--buffer: calc( 2 * var(--global-space--grid-gap) ); | ||
} | ||
|
||
@media only screen and (--medium-and-above) { | ||
|
@@ -24,10 +32,14 @@ Styleguide Objects.OffsetContent | |
margin-right: var(--buffer); | ||
} | ||
.o-offset-content--left + .o-offset-content--left { clear: left; } | ||
|
||
.figure:is(:--o-offset-content) .figure-caption { | ||
padding-right: 125px; | ||
} | ||
} | ||
@media only screen and (--medium-and-above) and (--max-wide-and-below) { | ||
[class*="o-offset-content--"] { | ||
max-width: 50%; | ||
:--o-offset-content { | ||
max-width: 560px; | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The offset content changes support the use of ECEP Articles Using this Pattern
|
||
/* Apply negative margin only when using offset value */ | ||
.o-offset-content--right { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hint: This imported stylesheet was added in #5.