Skip to content

PR #1 updatesToLanyardLandTheme#1

Open
rgadon107 wants to merge 20 commits intoDigitalCanvas:masterfrom
rgadon107:development
Open

PR #1 updatesToLanyardLandTheme#1
rgadon107 wants to merge 20 commits intoDigitalCanvas:masterfrom
rgadon107:development

Conversation

@rgadon107
Copy link

@rgadon107 rgadon107 commented Jun 11, 2020

PR Summary

This repository is a fork of the 'landyardland-theme' from DigitalCanvas. This PR contains the updates to the modified-genesis-sample child theme ( 'landyardland-theme' ) used to complete the development assignment for Digital Canvas LLC.

Directories and files targeted by this PR:

  • /config/layouts.php,
  • /config/appearance.php,
  • /lib/functions.php,
  • functions.php,
  • /lib/woocommerce/genesis-sample-woocommerce.css,
  • /sass/variables/_colors.scss,
  • /sass/variables/_typography.scss,
  • /sass/components/_list-items.scss,
  • /sass/_defaults.scss,
  • /sass/_plugins.scss, and
  • /sass/_content.scss.

Modifications to the WooCommerce plugin ( Task #2 ) can be viewed in the extend-woocommerce plugin at https://github.com/rgadon107/extend-woocommerce. Please look at the file /extend-woocommerce/src/asset/load-assets.php for the relevant code that manages the production description template.

rgadon107 added 20 commits June 9, 2020 14:03
- Return the layout configuration to 'full-width-content' only.
- clear floats on pseudo-elements.
- Add flexbox styles to container element.
-Add separate _colors and _typography partial files. 
- Declare variables for use throughout stylesheet.
- font-family;
- font-weight;
- font-size;
- Declare variables defined in 'variables' directory. Changes can be 
managed in one location rather than many.
- Includes specialized styling to color bullet points differently than 
list item elements.
- Set 'list-type-disc' property to 'none'. Prevent display of 2 sets of 
bullet items on same content when 'ul.color-with-psuedo' in use. 
- Remove float clearing on .woocommerce selector. Not needed. 
- Modify list-items partial file. Add right margin between bullet and 
<li> content. Fix top property to align bullets.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant