Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

Convert to SCSS Modules #253

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Convert to SCSS Modules #253

wants to merge 34 commits into from

Conversation

pacollins
Copy link
Owner

@pacollins pacollins commented May 8, 2021

Description ⚠️ WIP - Do not merge! ⚠️

Convert from a single SCSS file to SCSS modules to allow better customization and updates.

Changes

❌ Removed ❌

Templates (HTML)

  • partials/site-sidebar.html ❌ Removed (Merged into partials/site-footer.html)

CSS

  • nav#site-nav ❌ Removed
  • $breakpoints and @include break($breakpoint) ❌ Removed -up suffix (ex. $desktop-up -> $desktop)
  • #site-intro header container ❌ Removed
  • #site-intro main container ❌ Removed
  • #site-intro footer container ❌ Removed
  • .article-terms-link ❌ Removed
  • #recent-posts header container ❌ Removed
  • #recent-posts footer container ❌ Removed
  • #categories header container ❌ Removed
  • #categories footer container ❌ Removed

⚠️ Changed ⚠️

Templates (HTML)

  • partials/socnet-icon.html ⚠️ Changed to partials/social-icons.html

CSS

  • .nav .link ⚠️ Merged to .nav-link
  • menu#site-nav-menu.flyout-menu.menu ⚠️ Changed to nav#site-nav-menu.flyout-menu
  • .share-toggle. .search-toggle, .nav-toggle ⚠️ Seperated to .toggle + .share-button, .search-button, or .nav-button
  • #search (#search-input.active, #search-results.active) ⚠️ Moved .active to #search and ❌ Removed from #search-input and #search-results
  • .socnet-icons ⚠️ Changed to .social-icons
  • .title ⚠️ Changed to .post-description
  • .meta ⚠️ Changed to .post-meta
  • .content ⚠️ Changed to .post-content
  • section#site-sidebar ⚠️ Changed to footer#site-footer
  • #mini-bio ⚠️ Changed to #about
  • footer#site-footer ⚠️ Changed to section#copyright

Motivation and Context

[Why is this change required? What problem does it solve?]

Progress towards #252
Closes #250
Progress towards #244
Progress towards #218
Closes #164
Closes #178

Screenshots (if appropriate):

N/A

Checklist:

  • I have updated the documentation, as applicable.
  • I have updated the theme.toml, as applicable.

@pacollins
Copy link
Owner Author

Typography Update

image
image
image
image
image

Would love any feedback/suggestions as I keep trucking along.

@pacollins
Copy link
Owner Author

Adjusted the figure option to maintain fancybox shortcode, but move all new styles to gallery short code. Will allow for custom styling as well with type property.

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FEATURE: user-header.js and user-footer.js ENHANCEMENT: Update Typography
1 participant