Skip to content

Commit

Permalink
Merge pull request #10 from web-illinois/jonker/BugFixes
Browse files Browse the repository at this point in the history
Jonker/bug fixes
  • Loading branch information
bryanjonker-illinois authored Oct 8, 2024
2 parents aa3539c + aeb7664 commit c208901
Show file tree
Hide file tree
Showing 9 changed files with 45 additions and 22 deletions.
17 changes: 13 additions & 4 deletions builder/versions/ilw-content-inset.1.0-beta.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@
"element-name": "ilw-content",
"description": "The Content component with the inset mode enabled",
"builder-version": "1.0-beta",
"version": "1.0.0-beta",
"date": "9/13/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.0-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-hero/1.0.0-beta/ilw-content.js",
"version": "1.0.1-beta",
"date": "10/08/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.js",
"production": false,
"notes": "",
"parent-style": "margin: 0 auto; max-width: 1200px;",
Expand Down Expand Up @@ -70,6 +70,15 @@
"center",
"right"
]
},
{
"name": "transparent",
"description": "Whether the background is transparent -- note it will use the styling of whatever theme it is set to.",
"depreciated": false,
"values": [
"",
"true"
]
}
],
"classes-fixed": [],
Expand Down
8 changes: 4 additions & 4 deletions builder/versions/ilw-content-introduction.1.0-beta.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@
"element-name": "ilw-content",
"description": "The Content component with the introduction mode enabled. Note that the emphasized piece needs to be in an emphasized tag",
"builder-version": "1.0-beta",
"version": "1.0.0-beta",
"date": "9/13/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.0-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-hero/1.0.0-beta/ilw-content.js",
"version": "1.0.1-beta",
"date": "10/08/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.js",
"production": false,
"notes": "",
"parent-style": "margin: 0 auto; max-width: 1200px;",
Expand Down
8 changes: 4 additions & 4 deletions builder/versions/ilw-content-lede.1.0-beta.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@
"element-name": "ilw-content",
"description": "The Content component with the lede mode enabled",
"builder-version": "1.0-beta",
"version": "1.0.0-beta",
"date": "9/13/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.0-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-hero/1.0.0-beta/ilw-content.js",
"version": "1.0.1-beta",
"date": "10/08/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.js",
"production": false,
"notes": "",
"parent-style": "margin: 0 auto; max-width: 1200px;",
Expand Down
8 changes: 4 additions & 4 deletions builder/versions/ilw-content.1.0-beta.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@
"element-name": "ilw-content",
"description": "The Content component.",
"builder-version": "1.0-beta",
"version": "1.0.0-beta",
"date": "9/13/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.0-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-hero/1.0.0-beta/ilw-content.js",
"version": "1.0.1-beta",
"date": "10/08/2024",
"css": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.css",
"js": "https://dev.toolkit.illinois.edu/ilw-content/1.0.1-beta/ilw-content.js",
"production": false,
"notes": "",
"parent-style": "margin: 0 auto; max-width: 1200px;",
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"description": "Illinois Toolkit: Basic content item.",
"repository": "github:web-illinois/ilw-content",
"private": false,
"version": "1.0.0-beta",
"version": "1.0.1-beta",
"type": "module",
"files": [
"src/**",
Expand Down
15 changes: 14 additions & 1 deletion samples/inset.html
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@
if (params.has('theme')) {
component.setAttribute('theme', params.get('theme'));
}
if (params.has('transparent')) {
component.setAttribute('transparent', true);
}
if (params.has('width')) {
component.setAttribute('width', params.get('width'));
}
Expand All @@ -35,6 +38,16 @@ <h2>Lorem ipsum dolor sit amet</h2><p>Nunc sed leo arcu. Nulla non eleifend sapi
<h3>Lorem ipsum dolor sit amet</h3><p>Nunc sed leo arcu. Nulla non eleifend sapien. </p>
</ilw-content>
<ilw-content mode="inset">
<h4>Lorem ipsum dolor sit amet</h4><p>Nunc sed leo arcu. Nulla non eleifend sapien. </p>
<h3>Student Life</h3>
<p>
Animal sciences students extend their learning and career networks beyond
the classroom through internships, <a href="#">judging teams</a>, student
organizations, undergraduate research projects with our faculty, as well
as short- and long-term study abroad opportunities all over the world.
</p>
<ul class="ilw-buttons">
<li><a href="#">Subscribe</a></li>
<li><a href="#">Find a departmental advisor</a></li>
</ul>
</ilw-content>
</body>
4 changes: 2 additions & 2 deletions src/ilw-content.css
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,8 @@ ilw-content h6, ilw-content p.ilw-heading {
}

ilw-content h1 {
font-size: 4.25rem;
line-height: 5.25rem;
font-size: 3.25rem;
line-height: 4rem;
margin-top: 1.3125rem;
margin-bottom: 2.625rem;
font-family: var(--il-font-heading);
Expand Down
3 changes: 2 additions & 1 deletion src/ilw-content.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ class Content extends LitElement {
static get properties() {
return {
theme: { type: String, attribute: true },
transparent: { type: Boolean, attribute: true },
mode: { type: String, attribute: true },
align: { type: String, attribute: true },
width: { type: String, attribute: true },
Expand Down Expand Up @@ -42,7 +43,7 @@ class Content extends LitElement {

render() {
return html`
<div class="content ${this.mode} ${this.theme} ${this.outerWidth}">
<div class="content ${this.mode} ${this.theme} ${this.outerWidth} ${this.transparent ? 'transparent' : ''}">
<div style="${this.paddingStyle}" class="content-inner ${this.innerWidth} ${this.align}">
<div class="content-padding">
<slot></slot>
Expand Down
2 changes: 1 addition & 1 deletion src/ilw-content.styles.js
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ export default css`
--ilw-content--background-color: var(--il-storm-lighter-4);
}
.content.inset {
.content.transparent {
--ilw-content--background-color: transparent;
}
Expand Down

0 comments on commit c208901

Please sign in to comment.