Comments (6)
Thanks for starting this discussion! The main place for possible upstream changes would be in _sass/custom/custom.scss
where I have some stylistic overrides to provide different defaults for HTML code
, iframe
, details
, and summary
.
This reminded me that I also need to backport changes from the upstream _layouts/default.html
to my _layouts/minimal.html
. It might be worth considering maintaining this upstream if we see the use case for a layout for special pages that should not display the side and top navigation. I use it for my primarily single-page websites, but I could also see it helpful for some special standalone pages (perhaps marketing, events, etc that aren't as connected to the primary navigation).
If this seems interesting, I could open an issue or PR upstream to work through the engineering or design considerations.
from just-the-class.
Thanks for the quick response Kevin!
The main place for possible upstream changes would be in _sass/custom/custom.scss where I have some stylistic overrides to provide different defaults for HTML code, iframe, details, and summary.
Gotcha - let me take a look at those and see what would make sense to bring upstream.
It might be worth considering maintaining this upstream if we see the use case for a layout for special pages that should not display the side and top navigation.
Oh, interesting! I think there's certainly a use-case for this sort of page / layout. If we make it upstream, we can likely have default
consume it, which should lower code duplication on your end (as well as other users). If you have the time, feel free to open an issue and/or PR to discuss!
Independently, I'm TAing a class at UCLA this fall quarter and will give just-the-class a spin. Should make me more familiar with the theme + better understand how to support the theme moving forward! Great work with creating and maintaining it so far 😊
from just-the-class.
Thanks! I just opened just-the-docs/just-the-docs#959 for discussion.
from just-the-class.
Just to circle back on the styles in custom.scss
, I had a chance to try each of them and see which ones we can upstream. Some of them I think are easily doable, others not so much. Apologies for the massive wall of text!
massive wall of text :(
code
: I think this may be too big of a stylistic change for the core of the theme :(
code {
font-size: 14px;
padding: 0.2em 0.4em;
border: none;
}
details
: this one is interesting. I think in general, I could see this being quite useful, but we currently rely on details
having margin: 0
since it functions as a collapser for the table of contents - see the navigation structure page's TOC. I do see this being useful as a class - would adding support there be helpful?
details {
margin: 0 40px 1em;
}
iframe
/summary
: this is quite reasonable to me! In general a fan of max-width: 100%
default :)
iframe,
summary {
max-width: 100%;
}
summary
: for the same reason as details
, this breaks the existing styling. Would a class upstream for this be helpful? Or a mixin-sort of override that is only @extend .btn, .btn-outline;
?
summary {
@extend .btn, .btn-outline;
}
.main-content-wrap
: could you explain where you use this? I'm doing a visual diff on our site and I don't see any layout changes (which maybe, is a good thing!). Is this for sites that you override with the minimal layout?
.main-content-wrap {
max-width: $content-width;
margin: auto;
}
.main-content > dl
/ dt
: I think the font weight and text-align overrides are probably upstreamable, though the grid-first behaviour (which was introduced a long time ago!!) is probably desired by some users. Maybe a class here?
.main-content {
dl {
display: block;
grid-template-columns: none;
}
dt {
font-weight: 700;
text-align: start;
&::after {
content: normal;
}
}
dd {
font-weight: normal;
+ dt {
margin-top: 1em;
}
}
}
upstreaming this
If you feel comfortable opening a PR, you should go for it - you did write this code! If it's easier for me to integrate these changes (I know the quarter is starting soon, so totally understandable if you don't have much time) I can take a look at doing this myself. Let me know what you think!
from just-the-class.
Yeah, .main-content-wrap
is needed for the minimal layouts the way I designed div
s in the layout.
The grid format for definition lists (dl
, dt
, dd
) work fine for short definition terms, but longer definition terms (phrases or sentences) didn't work well because they would push the content too far to the side. Using block format is better for these longer phrases.
from just-the-class.
FYI - just released rc4
!
I know the new minimal layout is not a drop-in replacement for your current one, but hopefully at least you now don't need to copy over all the components. I can take a look at getting search to work as well (with your follow-up issue) - if you have bandwidth to take that on before the quarter starts though, go for it!
from just-the-class.
Related Issues (12)
- Course Logo HOT 1
- Site not displaying correctly HOT 2
- Unable to display more than one section on the same day HOT 3
- Linking to other site pages HOT 2
- Dark Mode Toggle HOT 1
- Controlling the order in which the modules appear HOT 1
- Overlapping schedule items HOT 2
- Staff photos stretched out HOT 1
- delete HOT 1
- Visualization of website not correct HOT 1
- README inaccurate instructions for local dev
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from just-the-class.